Re: pgsql: Make the order of the header file includes consistent in backend

From: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
To: Amit Kapila <akapila(at)postgresql(dot)org>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Make the order of the header file includes consistent in backend
Date: 2019-11-12 11:49:20
Message-ID: CAA4eK1L0LXwkNVbkWGy6e_HbQU_DKaK-yuA5zNvUtSEm+mVEuQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Tue, Nov 12, 2019 at 8:33 AM Amit Kapila <akapila(at)postgresql(dot)org> wrote:
>
> Make the order of the header file includes consistent in backend modules.
>

I see two buildfarm failures at this commit, but none seems to related
to this change.

1. The failure of anole seems to have happened earlier as well.
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=anole&dt=2019-11-12%2005%3A43%3A59
--today's failure
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=anole&dt=2019-07-18%2021%3A57%3A59
-- previous failure

2. The failure of fairywren seems somewhat random.
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=fairywren&dt=2019-11-12%2005%3A57%3A00

The log shows:

not ok 44 - pgbench simple update stdout /(?^:builtin: simple update)/

# Failed test 'pgbench simple update stdout /(?^:builtin: simple update)/'
# at t/001_pgbench_with_server.pl line 162.
# ''
# doesn't match '(?^:builtin: simple update)'
not ok 45 - pgbench simple update stdout /(?^:clients: 5\b)/

# Failed test 'pgbench simple update stdout /(?^:clients: 5\b)/'
# at t/001_pgbench_with_server.pl line 162.
# ''
# doesn't match '(?^:clients: 5\b)'

It is not clear to me how it can be related to this commit. If
anybody thinks so, please let me know. I will wait for another run
and see if this gets reproduced.

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-11-12 18:00:19 pgsql: Fix ecpglib.h to declare bool consistently with c.h.
Previous Message Michael Paquier 2019-11-12 08:48:09 Re: pgsql: gitattributes: Add new file