From: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
---|---|
To: | movead(dot)li(at)highgo(dot)ca |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Asynchronous Append on postgres_fdw nodes. |
Date: | 2020-03-10 07:50:37 |
Message-ID: | 20200310.165037.1952988224985652604.horikyota.ntt@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello. Thank you for testing.
At Tue, 10 Mar 2020 05:13:42 +0000, movead li <movead(dot)li(at)highgo(dot)ca> wrote in
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, failed
> Implements feature: tested, passed
> Spec compliant: tested, passed
> Documentation: not tested
>
> I occur a strange issue when a exec 'make installcheck-world', it is:
I had't done that.. Bud it worked for me.
> ##########################################################
> ...
> ============== running regression test queries ==============
> test adminpack ... FAILED 60 ms
>
> ======================
> 1 of 1 tests failed.
> ======================
>
> The differences that caused some tests to fail can be viewed in the
> file "/work/src/postgres_app_for/contrib/adminpack/regression.diffs". A copy of the test summary that you see
> above is saved in the file "/work/src/postgres_app_for/contrib/adminpack/regression.out".
> ...
> ##########################################################
>
> And the content in 'contrib/adminpack/regression.out' is:
I don't see that file. Maybe regression.diff?
> ##########################################################
> SELECT pg_file_write('/tmp/test_file0', 'test0', false);
> ERROR: absolute path not allowed
> SELECT pg_file_write(current_setting('data_directory') || '/test_file4', 'test4', false);
> - pg_file_write
> ----------------
> - 5
> -(1 row)
> -
> +ERROR: reference to parent directory ("..") not allowed
It seems to me that you are setting a path containing ".." to PGDATA.
> However the issue does not occur when I do a 'make check-world'.
> And it doesn't occur when I test the 'make installcheck-world' without the patch.
check-world doesn't use path containing ".." as PGDATA.
> The new status of this patch is: Waiting on Author
Thanks for noticing that.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2020-03-10 08:01:43 | Re: reindex concurrently and two toast indexes |
Previous Message | Kyotaro Horiguchi | 2020-03-10 06:17:23 | Re: Crash by targetted recovery |