From: | Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | Ryan Murphy <ryanfmurphy(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: postgres_fdw bug in 9.6 |
Date: | 2017-09-05 07:40:21 |
Message-ID: | 07ed8f27-12cf-8403-d23b-f7d907de4a83@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2017/09/05 13:43, Ryan Murphy wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, passed
> Implements feature: not tested
> Spec compliant: not tested
> Documentation: not tested
>
> This feature is obviously a pretty deep cut, and I don't understand the JOIN system enough to comment on whether the code is correct. I did not see any issues when glancing through the patch.
Thank you for the review!
> I ran "make check", "make installcheck" and "make installcheck-world" and had ALMOST no problems: I marked it Tested and Passed because the only issue I had was a recurring issue I've had with "make installcheck-world" which I think is unrelated:
>
> *** path/to/postgres/src/interfaces/ecpg/test/expected/connect-test5.stderr 2017-02-14 09:22:25.000000000 -0600
> --- path/to/postgres/src/interfaces/ecpg/test/results/connect-test5.stderr 2017-09-04 23:31:50.000000000 -0500
> ***************
> *** 36,42 ****
> [NO_PID]: sqlca: code: 0, state: 00000
> [NO_PID]: ECPGconnect: opening database <DEFAULT> on <DEFAULT> port <DEFAULT> for user regress_ecpg_user2
> [NO_PID]: sqlca: code: 0, state: 00000
> ! [NO_PID]: ECPGconnect: could not open database: FATAL: database "regress_ecpg_user2" does not exist
> ...
I tested that with HEAD, but couldn't reproduce this problem. Didn't
you test that with HEAD?
> But this still Needs Review by someone who knows the JOIN system better.
I think Tom is reviewing this patch [1].
Best regards,
Etsuro Fujita
[1] https://www.postgresql.org/message-id/22168.1504041271%40sss.pgh.pa.us
From | Date | Subject | |
---|---|---|---|
Next Message | Surafel Temesgen | 2017-09-05 07:45:55 | Re: Support to COMMENT ON DATABASE CURRENT_DATABASE |
Previous Message | Masahiko Sawada | 2017-09-05 07:24:04 | Re: pgbench: Skipping the creating primary keys after initialization |