Hello hackers, I've been a reader of this list for some time, but have never posted.
I have interest in the URI connection string support patch[1], so I'm in the process of reviewing it. I have a couple of comments and questions: 1. I see no tests in the patch. I'd like to start getting together a set of tests, likely based on the connection string permutations found on Greg Smith's response[2]. However I don't find an obvious place to put them. They could possibly live in the test/examples directory. Another thought is to use dblink in a test, although it may be problematic to depend on a contrib package for a test, to say the least. Any thoughts on how to test this are most welcome. 2. The documentation/manual was not updated as part of this patch, so this is pending. 3. I for one do prefer the `postgres` prefix, as opposed to `postgresql` for the reasons stated on an earlier thread [3]. In my opinion the best way to move forward is to support them both. The good news is the patch still applies fine on the 9.2 HEAD, and seems to work well. Regards, -Harold [1] https://commitfest.postgresql.org/action/patch_view?id=720 [2] http://archives.postgresql.org/message-id/4f45253c.4030...@2ndquadrant.com [3] http://archives.postgresql.org/pgsql-hackers/2011-12/msg00499.php
