pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
pgsql: Put enable_self_join_elimination into postgresql.conf.sample
3+ messages / 3 participants
[nested] [flat]

Commits from this thread:
  2025-04-06 3a1a7c5a70  Revert "Put enable_self_join_elimination into postgresql.conf.sample"
* pgsql: Put enable_self_join_elimination into postgresql.conf.sample
@ 2025-04-06 10:27 Alexander Korotkov <akorotkov@postgresql.org>
  2025-04-06 11:10 ` Re: pgsql: Put enable_self_join_elimination into postgresql.conf.sample Daniel Gustafsson <daniel@yesql.se>
  0 siblings, 1 reply; 3+ messages in thread

From: Alexander Korotkov @ 2025-04-06 10:27 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

Put enable_self_join_elimination into postgresql.conf.sample

fc069a3a6319 implements Self-Join Elimination (SJE) and provides a new
GUC variable: enable_self_join_elimination.  This commit adds
enable_self_join_elimination to the postgresql.conf.sample, as it was
forgotten in the original commit.

Discussion: https://postgr.es/m/CAHewXN%3D%2Bghd6O6im46q7j2u6c3H6vkXtXmF%3D_v4CfGSnjje8PA%40mail.gmail.com
Author: Tender Wang <tndrwang@gmail.com>

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c2d329260cd885a937014cecd38b51bdeae65216

Modified Files
--------------
src/backend/utils/misc/postgresql.conf.sample | 1 +
1 file changed, 1 insertion(+)



^ permalink  raw  reply  [nested|flat] 3+ messages in thread

* Re: pgsql: Put enable_self_join_elimination into postgresql.conf.sample
  2025-04-06 10:27 pgsql: Put enable_self_join_elimination into postgresql.conf.sample Alexander Korotkov <akorotkov@postgresql.org>
@ 2025-04-06 11:10 ` Daniel Gustafsson <daniel@yesql.se>
  2025-04-06 11:30   ` Re: pgsql: Put enable_self_join_elimination into postgresql.conf.sample Alexander Korotkov <aekorotkov@gmail.com>
  0 siblings, 1 reply; 3+ messages in thread

From: Daniel Gustafsson @ 2025-04-06 11:10 UTC (permalink / raw)
  To: Alexander Korotkov <akorotkov@postgresql.org>; +Cc: pgsql-committers@lists.postgresql.org

> On 6 Apr 2025, at 12:27, Alexander Korotkov <akorotkov@postgresql.org> wrote:
> 
> Put enable_self_join_elimination into postgresql.conf.sample
> 
> fc069a3a6319 implements Self-Join Elimination (SJE) and provides a new
> GUC variable: enable_self_join_elimination.  This commit adds
> enable_self_join_elimination to the postgresql.conf.sample, as it was
> forgotten in the original commit.

The GUC is marked GUC_NOT_IN_SAMPLE: 

+       {"enable_self_join_elimination", PGC_USERSET, QUERY_TUNING_METHOD,
+           gettext_noop("Enable removal of unique self-joins."),
+           NULL,
+           GUC_EXPLAIN | GUC_NOT_IN_SAMPLE
+       },

..which is causing an error in the buildfarm:

# Failed test 'no parameters missing from guc_tables.c'
# at t/003_check_guc.pl line 89.
# got: '1'
# expected: '0'

# Failed test 'no parameters marked as NOT_IN_SAMPLE in postgresql.conf.sample'
# at t/003_check_guc.pl line 92.
# got: '1'
# expected: '0'
# Looks like you failed 2 tests of 3.

--
Daniel Gustafsson






^ permalink  raw  reply  [nested|flat] 3+ messages in thread

* Re: pgsql: Put enable_self_join_elimination into postgresql.conf.sample
  2025-04-06 10:27 pgsql: Put enable_self_join_elimination into postgresql.conf.sample Alexander Korotkov <akorotkov@postgresql.org>
  2025-04-06 11:10 ` Re: pgsql: Put enable_self_join_elimination into postgresql.conf.sample Daniel Gustafsson <daniel@yesql.se>
@ 2025-04-06 11:30   ` Alexander Korotkov <aekorotkov@gmail.com>
  0 siblings, 0 replies; 3+ messages in thread

From: Alexander Korotkov @ 2025-04-06 11:30 UTC (permalink / raw)
  To: Daniel Gustafsson <daniel@yesql.se>; +Cc: Alexander Korotkov <akorotkov@postgresql.org>; pgsql-committers@lists.postgresql.org

On Sun, Apr 6, 2025 at 2:10 PM Daniel Gustafsson <daniel@yesql.se> wrote:
>
> > On 6 Apr 2025, at 12:27, Alexander Korotkov <akorotkov@postgresql.org> wrote:
> >
> > Put enable_self_join_elimination into postgresql.conf.sample
> >
> > fc069a3a6319 implements Self-Join Elimination (SJE) and provides a new
> > GUC variable: enable_self_join_elimination.  This commit adds
> > enable_self_join_elimination to the postgresql.conf.sample, as it was
> > forgotten in the original commit.
>
> The GUC is marked GUC_NOT_IN_SAMPLE:
>
> +       {"enable_self_join_elimination", PGC_USERSET, QUERY_TUNING_METHOD,
> +           gettext_noop("Enable removal of unique self-joins."),
> +           NULL,
> +           GUC_EXPLAIN | GUC_NOT_IN_SAMPLE
> +       },
>
> ..which is causing an error in the buildfarm:
>
> # Failed test 'no parameters missing from guc_tables.c'
> # at t/003_check_guc.pl line 89.
> # got: '1'
> # expected: '0'
>
> # Failed test 'no parameters marked as NOT_IN_SAMPLE in postgresql.conf.sample'
> # at t/003_check_guc.pl line 92.
> # got: '1'
> # expected: '0'
> # Looks like you failed 2 tests of 3.

Ooops, I'm going to revert this now.

------
Regards,
Alexander Korotkov
Supabase





^ permalink  raw  reply  [nested|flat] 3+ messages in thread


end of thread, other threads:[~2025-04-06 11:30 UTC | newest]

Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2025-04-06 10:27 pgsql: Put enable_self_join_elimination into postgresql.conf.sample Alexander Korotkov <akorotkov@postgresql.org>
2025-04-06 11:10 ` Daniel Gustafsson <daniel@yesql.se>
2025-04-06 11:30   ` Alexander Korotkov <aekorotkov@gmail.com>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox