Re: Tab completion regression test failed on illumos - Mailing list pgsql-hackers

From Japin Li
Subject Re: Tab completion regression test failed on illumos
Date
Msg-id MEYP282MB1669A580A62C2CDE30951E81B6A6A@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM
Whole thread Raw
In response to Re: Tab completion regression test failed on illumos  (Japin Li <japinli@hotmail.com>)
Responses Re: Tab completion regression test failed on illumos
List pgsql-hackers
On Thu, 02 Nov 2023 at 13:42, Japin Li <japinli@hotmail.com> wrote:
> On Thu, 02 Nov 2023 at 13:01, Noah Misch <noah@leadboat.com> wrote:
>> On Wed, Nov 01, 2023 at 03:19:39PM +0800, Japin Li wrote:
>>> I try to run regression test on illumos, the 010_tab_completion will
>>> failed because of timeout.
>>
>>> Any suggestions?  Thanks in advance!
>>
>> This test failed for me, in a different way, when I briefly installed IO::Pty
>> on a Solaris buildfarm member:
>> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=wrasse&dt=2023-01-03%2022%3A39%3A26
>>
> Thanks for confirm this!
>
> I try to install IO::Pty using cpan, however, I cannot get the same error.
>
>> The IPC::Run pty tests also fail on Solaris.  If I were debugging this, I'd
>> start by fixing IO::Pty and IPC::Run to pass their own test suites on Solaris
>> or illumos.  Then I'd see if problems continue for this postgresql test.
>
> So, it might be a bug comes from Perl.

After enable debug for IPC::Run, I found the following logs:

IPC::Run 0001 0123456789012-4 [#2(415745)]: writing to fd 11 (kid's stdin)
IPC::Run 0001 0123456789012-4 [#2(415745)]: write( 11, 'SEL     ' ) = 4             <- Here write 4 bytes.
IPC::Run 0001 0123456789012-4 [#2(415745)]: fds for select: -----------r--r
IPC::Run 0001 0123456789012-4 [#2(415745)]: timeout=0
IPC::Run 0001 0123456789012-4 [#2(415745)]: selected  -----------r
IPC::Run 0001 0123456789012-4 [#2(415745)]: filtering data from fd 11 (kid's stdout)
IPC::Run 0001 0123456789012-4 [#2(415745)]: reading from fd 11 (kid's stdout)
IPC::Run 0001 0123456789012-4 [#2(415745)]: read( 11 ) = 8 chars 'SEL     '         <- But read 8 bytes.

It seems the 'SEL\t' is converted to 'SEL     ' which is "SEL" with 5 spaces.

-- 
Regrads,
Japin Li
ChengDu WenWu Information Technology Co., Ltd.



pgsql-hackers by date:

Previous
From: Crisp Lee
Date:
Subject: make pg_ctl more friendly
Next
From: David Rowley
Date:
Subject: Re: Why is DEFAULT_FDW_TUPLE_COST so insanely low?