Thanks for this. I have tried out your patch, and it works as well as the original in our test set. But the test set does not test the Perl interface very well. There are just a few tests, but they work.
I'll commit this to the Oct2014 branch since it does seem to fix bugs.
As to the best place to send and/or discuss pathes, I would say the bug tracker. However, I do realize that the developer list may well be read by more people. The bug tracker has the advantage that patches and the discussion about them don't get lost.
On 12/01/15 22:24, Stefan O'Rear wrote:
I have been integrating MonetDB into a Perl system for the last few months, and have made a few changes to the Perl driver to improve performance and reliability.
I hereby release these changes under the same license as MonetDB itself. I have permission from my employer to do this.
Is this to correct place to send and/or discuss patches?
In order of patch hunks; more details will be provided for any part if requested:
- removes "use sigtrap", because this has global effects and should
not be used by modules, only by the application. 2. allows Perl 5.8.1+ Unicode strings to be passed to quote() and included in statements (UTF-8 encoded, as expected by Monet's str module) 3. quote and unquote now use the same quoting rules as the MonetDB server, allowing for all characters except NUL to be round-tripped 4. several character loops have been reimplemented in regex for much greater performance 5. micro-optimizations to the result fetch loop 6. block boundaries are preserved in piggyback data so that Xclose is not appended or prepended to a SQL command 7. diagnostic messages #foo before a result header are ignored, this is necessary to use recycler_pipe 8. fail quickly and loudly if we receive a continuation prompt (or any other response that starts with a non-ASCII character) 9. header lines must start with %, not merely contain %, fixing a bug when querying a table where string values contain % 10. after closing a large resultset, account for the fact that a reply will come and do not lose sync 11. allow a MAPI_TRACE environment variable to dump wire protocol frames to standard output 12. fixes maximum MAPI block size to match the server limit of 16k. previously would crash on blocks larger than 16k
developers-list mailing list developers-list@monetdb.org https://www.monetdb.org/mailman/listinfo/developers-list