Patches that don't apply or don't compile: 2017-09-12

From: Aleksander Alekseev <a(dot)alekseev(at)postgrespro(dot)ru>
To: pgsql-hackers(at)postgresql(dot)org
Cc: "Bossart, Nathan" <bossartn(at)amazon(dot)com>, "yangjie(at)highgo(dot)com" <yangjie(at)highgo(dot)com>, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, Amit Khandekar <amitdkhan(dot)pg(at)gmail(dot)com>, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Anton Dignös <dignoes(at)inf(dot)unibz(dot)it>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Elvis Pranskevichus <elprans(at)gmail(dot)com>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Jeff Davis <pgsql(at)j-davis(dot)com>, Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, Mark Rofail <markm(dot)rofail(at)gmail(dot)com>, Martijn van Oosterhout <kleptog(at)svana(dot)org>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, Pierre Ducroquet <p(dot)psql(at)pinaraf(dot)info>, Rafia Sabih <rafia(dot)sabih(at)enterprisedb(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Sokolov Yura <funny(dot)falcon(at)postgrespro(dot)ru>, Tianzhou Chen <tianzhouchen(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Victor Drobny <v(dot)drobny(at)postgrespro(dot)ru>, Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>
Subject: Patches that don't apply or don't compile: 2017-09-12
Date: 2017-09-12 14:14:55
Message-ID: 20170912141455.GB20517@e733.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello, hackers!

Thanks to the work of Thomas Munro now we have a CI for the patches on
the commitfest [1]. Naturally there is still room for improvement, but
in any case it's much, much better than nothing.

After a short discussion [2] we agreed (or at least no one objected) to
determine the patches that don't apply / don't compile / don't pass
regression tests and have "Needs Review" status, change the status of
these patches to "Waiting on Author" and write a report (this one) with
a CC to the authors. As all we know, we are short on reviewers and this
action will save them a lot of time. Here [3] you can find a script I've
been using to find such patches.

I rechecked the list manually and did my best to exclude the patches
that were updated recently or that depend on other patches. However
there still a chance that your patch got to the list by a mistake. In
this case please let me know.

Unless there are any objections I'm going to re-run this script once or
twice a week during the commitfest and write reports like this one.

Here is the list:

=== Apply Failed: 25 ===
Title: 64-bit transaction identifiers
Author: Tianzhou Chen <tianzhouchen(at)gmail(dot)com>
(it's a bug - actually Alexander Korotkov)
URL: https://commitfest.postgresql.org/14/1178/

Title: Add and report the new "in_hot_standby" GUC pseudo-variable.
Author: Elvis Pranskevichus <elprans(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1090/

Title: allow mix of composite and scalar variables in target list
Author: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1139/

Title: Allow users to specify multiple tables in VACUUM commands
Author: "Bossart, Nathan" <bossartn(at)amazon(dot)com>
URL: https://commitfest.postgresql.org/14/1131/

Title: Controlling toast_tuple_target to tune rows >2kB
Author: Simon Riggs <simon(at)2ndquadrant(dot)com>
URL: https://commitfest.postgresql.org/14/1284/

Title: Convert join OR clauses into UNION queries
Author: Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>
URL: https://commitfest.postgresql.org/14/1001/

Title: faster partition pruning in planner
Author: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>
URL: https://commitfest.postgresql.org/14/1272/

Title: Fix race condition between SELECT and ALTER TABLE NO INHERIT
Author: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
URL: https://commitfest.postgresql.org/14/1259/

Title: Gather speed-up
Author: Rafia Sabih <rafia(dot)sabih(at)enterprisedb(dot)com>
URL: https://commitfest.postgresql.org/14/1156/

Title: Hash partitioning
Authors: Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, "yangjie(at)highgo(dot)com" <yangjie(at)highgo(dot)com>
URL: https://commitfest.postgresql.org/14/1059/

Title: Improve compactify_tuples and PageRepairFragmentation
Author: Sokolov Yura <funny(dot)falcon(at)postgrespro(dot)ru>
URL: https://commitfest.postgresql.org/14/1138/

Title: Improve eval_const_expressions
Author: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
URL: https://commitfest.postgresql.org/14/1136/

Title: Incremental sort
Author: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>
URL: https://commitfest.postgresql.org/14/1124/

Title: libpq: Support for Apple Secure Transport SSL library
Author: Daniel Gustafsson <daniel(at)yesql(dot)se>
URL: https://commitfest.postgresql.org/14/1228/

Title: multivariate MCV lists and histograms
Author: Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>
URL: https://commitfest.postgresql.org/14/1238/

Title: New function for tsquery creation
Author: Victor Drobny <v(dot)drobny(at)postgrespro(dot)ru>
URL: https://commitfest.postgresql.org/14/1202/

Title: pg_basebackup has stricter tablespace rules than the server
Author: Pierre Ducroquet <p(dot)psql(at)pinaraf(dot)info>
URL: https://commitfest.postgresql.org/14/1125/

Title: Range Merge Join
Author: Jeff Davis <pgsql(at)j-davis(dot)com>
URL: https://commitfest.postgresql.org/14/1106/

Title: Retire polyphase merge
Author: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
URL: https://commitfest.postgresql.org/14/1267/

Title: Subscription code improvements
Author: Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>
URL: https://commitfest.postgresql.org/14/1248/

Title: Support arrays over domain types
Author: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
URL: https://commitfest.postgresql.org/14/1235/

Title: Support target_session_attrs=read-only and eliminate the added round-trip to detect session status.
Author: Elvis Pranskevichus <elprans(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1148/

Title: taking stdbool.h into use
Author: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
URL: https://commitfest.postgresql.org/14/1242/

Title: UPDATE of partition key
Author: Amit Khandekar <amitdkhan(dot)pg(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1023/

Title: Write Amplification Reduction Method (WARM)
Author: Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/775/

=== Build Failed: 7 ===
Title: Fix the optimization to skip WAL-logging on table created in same transaction
Author: Martijn van Oosterhout <kleptog(at)svana(dot)org>
URL: https://commitfest.postgresql.org/14/528/

Title: Foreign Key Arrays
Author: Mark Rofail <markm(dot)rofail(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1252/

Title: Generic type subscripting
Author: Dmitry Dolgov <9erthalion6(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1062/

Title: new plpgsql extra_checks
Author: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/962/

Title: Replication status in logical replication
Author: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
URL: https://commitfest.postgresql.org/14/1113/

Title: Restricting maximum keep segments by repslots
Author: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
URL: https://commitfest.postgresql.org/14/1260/

Title: Temporal query processing with range types
Author: Anton Dignös <dignoes(at)inf(dot)unibz(dot)it>
URL: https://commitfest.postgresql.org/14/839/

Needs Review Total: 120
Failed Total: 32 (26.67 %)

As always, any feedback is very welcomed!

[1]: http://commitfest.cputube.org/
[2]: https://postgr.es/m/CAB7nPqSrHF%2BkNQ6Eq2uy91RcysoCzQr1AjOjkuCn9jvMdJZ6Fw%40mail.gmail.com
[3]: https://github.com/afiskon/py-autoreviewer

--
Best regards,
Aleksander Alekseev

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2017-09-12 14:16:22 Re: Re: [COMMITTERS] pgsql: Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition b
Previous Message Tom Lane 2017-09-12 13:58:37 Re: Automatic testing of patches in commit fest