On Sun, 29 Jan 2023 at 22:05, vignesh C <vignes...@gmail.com> wrote: > > On Sun, 22 Jan 2023 at 22:00, vignesh C <vignes...@gmail.com> wrote: > > > > On Sun, 15 Jan 2023 at 23:02, vignesh C <vignes...@gmail.com> wrote: > > > > > > On Sun, 8 Jan 2023 at 21:00, vignesh C <vignes...@gmail.com> wrote: > > > > > > > > On Tue, 3 Jan 2023 at 13:13, vignesh C <vignes...@gmail.com> wrote: > > > > > > > > > > Hi All, > > > > > > > > > > Just a reminder that Commitfest 2023-01 has started. > > > > > There are many patches based on the latest run from [1] which require > > > > > a) Rebased on top of head b) Fix compilation failures c) Fix test > > > > > failure, please have a look and rebase it so that it is easy for the > > > > > reviewers and committers: > > > > > 1. TAP output format for pg_regress > > > > > 2. Add BufFileRead variants with short read and EOF detection > > > > > 3. Add SHELL_EXIT_CODE variable to psql > > > > > 4. Add foreign-server health checks infrastructure > > > > > 5. Add last_vacuum_index_scans in pg_stat_all_tables > > > > > 6. Add index scan progress to pg_stat_progress_vacuum > > > > > 7. Add the ability to limit the amount of memory that can be allocated > > > > > to backends. > > > > > 8. Add tracking of backend memory allocated to pg_stat_activity > > > > > 9. CAST( ... ON DEFAULT) > > > > > 10. CF App: add "Returned: Needs more interest" close status > > > > > 11. CI and test improvements > > > > > 12. Cygwin cleanup > > > > > 13. Expand character set for ltree labels > > > > > 14. Fix tab completion MERGE > > > > > 15. Force streaming every change in logical decoding > > > > > 16. More scalable multixacts buffers and locking > > > > > 17. Move SLRU data into the regular buffer pool > > > > > 18. Move extraUpdatedCols out of RangeTblEntry > > > > > 19.New [relation] options engine > > > > > 20. Optimizing Node Files Support > > > > > 21. PGDOCS - Stats views and functions not in order? > > > > > 22. POC: Lock updated tuples in tuple_update() and tuple_delete() > > > > > 23. Parallelize correlated subqueries that execute within each worker > > > > > 24. Pluggable toaster > > > > > 25. Prefetch the next tuple's memory during seqscans > > > > > 26. Pulling up direct-correlated ANY_SUBLINK > > > > > 27. Push aggregation down to base relations and joins > > > > > 28. Reduce timing overhead of EXPLAIN ANALYZE using rdtsc > > > > > 29. Refactor relation extension, faster COPY > > > > > 30. Remove NEW placeholder entry from stored view query range table > > > > > 31. TDE key management patches > > > > > 32. Use AF_UNIX for tests on Windows (ie drop fallback TCP code) > > > > > 33. Windows filesystem support improvements > > > > > 34. making relfilenodes 56 bit > > > > > 35. postgres_fdw: commit remote (sub)transactions in parallel during > > > > > pre-commit > > > > > 36.recovery modules > > > > > > > > > > Commitfest status as of now: > > > > > Needs review: 177 > > > > > Waiting on Author: 47 > > > > > Ready for Committer: 20 > > > > > Committed: 31 > > > > > Withdrawn: 4 > > > > > Rejected: 0 > > > > > Returned with Feedback: 0 > > > > > Total: 279 > > > > > > > > > > We will be needing more members to actively review the patches to get > > > > > more patches to the committed state. I would like to remind you that > > > > > each patch submitter is expected to review at least one patch from > > > > > another submitter during the CommitFest, those members who have not > > > > > picked up patch for review please pick someone else's patch to review > > > > > as soon as you can. > > > > > I'll send out reminders this week to get your patches rebased and > > > > > update the status of the patch accordingly. > > > > > > > > > > [1] - http://cfbot.cputube.org/ > > > > > > > > Hi Hackers, > > > > > > > > Here's a quick status report after the first week (I think only about > > > > 9 commits happened during the week, the rest were pre-CF activity): > > > > > > > > status | 3rd Jan | w1 > > > > -------------------------+-----------+----- > > > > Needs review: | 177 | 149 > > > > Waiting on Author: | 47 | 60 > > > > Ready for Committer: | 20 | 23 > > > > Committed: | 31 | 40 > > > > Withdrawn: | 4 | 7 > > > > Rejected: | 0 | 0 > > > > Returned with Feedback: | 0 | 0 > > > > Total: | 279 | 279 > > > > > > > > Here is a list of "Needs review" entries for which there has not been > > > > much communication on the thread and needs help in proceeding further. > > > > Please pick one of these and help us on how to proceed further: > > > > pgbench: using prepared BEGIN statement in a pipeline could cause an > > > > error | Yugo Nagata > > > > Fix dsa_free() to re-bin segment | Dongming Liu > > > > pg_rewind: warn when checkpoint hasn't happened after promotion | James > > > > Coleman > > > > Work around non-atomic read of read of control file on ext4 | Thomas > > > > Munro > > > > Rethinking the implementation of ts_headline | Tom Lane > > > > Fix GetWALAvailability function code comments for WALAVAIL_REMOVED > > > > return value | sirisha chamarti > > > > Function to log backtrace of postgres processes | vignesh C, Bharath > > > > Rupireddy > > > > disallow HEAP_XMAX_COMMITTED and HEAP_XMAX_IS_LOCKED_ONLY | Nathan > > > > Bossart > > > > New hooks in the connection path | Bertrand Drouvot > > > > Check consistency of GUC defaults between .sample.conf and > > > > pg_settings.boot_val | Justin Pryzby > > > > Add <<none>> support to sepgsql_restorecon | Joe Conway > > > > pg_stat_statements and "IN" conditions | Dmitry Dolgov > > > > Patch to implement missing join selectivity estimation for range types > > > > | Zhicheng Luo, Maxime Schoemans, Diogo Repas, Mahmoud SAKR > > > > Operation log for major operations | Dmitry Koval > > > > Consider parallel for LATERAL subqueries having LIMIT/OFFSET | James > > > > Coleman > > > > Using each rel as both outer and inner for anti-joins | Richard Guo > > > > partIndexlist for partitioned tables uniqueness | Arne Roland > > > > In-place persistence change of a relation (fast ALTER TABLE ... SET > > > > LOGGED with wal_level=minimal) | Kyotaro Horiguchi > > > > Speed up releasing of locks | Andres Freund, David Rowley > > > > nbtree performance improvements through specialization on key shape | > > > > Matthias van de Meent > > > > Add sortsupport for range types and btree_gist | Christoph Heiss > > > > asynchronous execution support for Custom Scan | KaiGai Kohei, kazutaka > > > > onishi > > > > > > > > Here is a list of "Ready for Committer" entries for which there has > > > > not been much communication on the thread and needs help in proceeding > > > > further. If any of the committers has some time to spare, please help > > > > us on these: > > > > Fix assertion failure with barriers in parallel hash join | Thomas > > > > Munro, Melanie Plageman > > > > pg_dump - read data for some options from external file | Pavel Stehule > > > > Add non-blocking version of PQcancel | Jelte Fennema > > > > reduce impact of lengthy startup and checkpoint tasks | Nathan Bossart > > > > pg_stat_statements: Track statement entry timestamp | Andrei Zubkov > > > > Add Amcheck option for checking unique constraints in btree indexes | > > > > Maxim Orlov, Pavel Borisov, Anastasia Lubennikova > > > > Introduce a new view for checkpointer related stats | Bharath Rupireddy > > > > Parallel Hash Full Join | Melanie Plageman > > > > Use fadvise in wal replay | Kirill Reshke, Jakub Wartak > > > > pg_receivewal fail to streams when the partial file to write is not > > > > fully initialized present in the wal receiver directory | Bharath > > > > Rupireddy, SATYANARAYANA NARLAPURAM > > > > Let libpq reject unexpected authentication requests | Jacob Champion > > > > Support % wildcard in extension upgrade scripts | Sandro Santilli > > > > TAP output format for pg_regress | Daniel Gustafsson > > > > > > > > If you have submitted a patch and it's in "Waiting for author" state, > > > > please aim to get it to "Needs review" state soon if you can, as > > > > that's where people are most likely to be looking for things to > > > > review. > > > > > > > > I have pinged most threads that are in "Needs review" state and don't > > > > apply, compile warning-free, or pass check-world. I'll do some more > > > > of that sort of thing, and I'll highlight a different set of patches > > > > next week. > > > > > > Hi Hackers, > > > > > > Here's a quick status report after the second week, there has been 13 > > > entries which were committed in the last week: > > > > > > status | 3rd Jan | w1 | w2 > > > -------------------------+-----------+-------+----- > > > Needs review: | 177 | 149 | 128 > > > Waiting on Author: | 47 | 60 | 64 > > > Ready for Committer: | 20 | 23 | 26 > > > Committed: | 31 | 40 | 53 > > > Withdrawn: | 4 | 7 | 7 > > > Rejected: | 0 | 0 | 0 > > > Returned with Feedback: | 0 | 0 | 1 > > > Total: | 279 | 279 | 279 > > > > > > Here is a few different patches which "Needs review", please pick one > > > of these and help us in proceeding further: > > > 1) Add semi-join pushdown to postgres_fdw | Alexander Pyhalov > > > 2) pg_upgrade test failure | Thomas Munro > > > 3) Fix progress report of CREATE INDEX for nested partitioned tables | > > > Ilya Gladyshev > > > 4) Non-replayable WAL records through overflows and >MaxAllocSize > > > lengths | Matthias van de Meent > > > 5) Add sslmode "no-clientcert" to avoid auth failure in md5/scram > > > connections | Jim Jones > > > 6) Add SHELL_EXIT_CODE variable to psql | Corey Huinker > > > 7) Fix GetWALAvailability function code comments for WALAVAIL_REMOVED > > > return value | sirisha chamarthi > > > 8) New strategies for freezing, advancing relfrozenxid early | Peter > > > Geoghegan > > > 9) Lockless queue of waiters based on atomic operations for LWLock | > > > Alexander Korotkov, Pavel Borisov > > > 10) Refactor relation extension, faster COPY | Andres Freund > > > 11) Add system view tracking shared buffer actions | Melanie Plageman > > > 12) Add index scan progress to pg_stat_progress_vacuum | Sami Imseih > > > 13) HOT chain validation in verify_heapam() | Himanshu Upadhyaya > > > 14) Periodic burst growth of the checkpoint_req counter on replica. | > > > Anton Melnikov > > > 15) Add EXPLAIN option GENERIC_PLAN for parameterized queries | Laurenz > > > Albe > > > 16) More scalable multixacts buffers and locking | Kyotaro Horiguchi , > > > Andrey Borodin , Ivan Lazarev > > > 17) In-place persistence change of a relation (fast ALTER TABLE ... > > > SET LOGGED with wal_level=minimal) | Kyotaro Horiguchi > > > 18) Reducing planning time when tables have many partitions | Yuya Watari > > > 19) ALTER TABLE and CLUSTER fail to use a BulkInsertState for toast > > > tables | Justin Pryzby > > > 20) Reduce timing overhead of EXPLAIN ANALYZE using rdtsc | Andres > > > Freund, Lukas Fittl, David Geier > > > > > > Here is a few different patches which are in "Ready for Committer" > > > state, if any of the committers has some time to spare, please have a > > > look: > > > 1) Support load balancing in libpq | Jelte Fennema > > > 2) Use the system CA pool for certificate verification |Jacob > > > Champion, Thomas Habets > > > 3) PG DOCS - pub/sub - specifying optional parameters without values. > > > | Peter Smith > > > 4) Doc: Rework contrib appendix -- informative titles, tweaked > > > sentences | Karl Pinc > > > 5) Amcheck verification of GiST and GIN | Andrey Borodin, Heikki > > > Linnakangas, Grigory Kryachko > > > 6) Introduce a new view for checkpointer related stats | Bharath Rupireddy > > > 7) Faster pglz compression | Andrey Borodin, tinsane > > > 8) AcquireExecutorLocks() and run-time pruning | Amit Langote > > > 9) Parallel Aggregates for string_agg and array_agg | David Rowley > > > 10) Simplify standby state machine a bit in > > > WaitForWALToBecomeAvailable() | Bharath Rupireddy > > > > > > If you have submitted a patch and it's in "Waiting for author" state, > > > please aim to get it to "Needs review" state soon if you can, as > > > that's where people are most likely to be looking for things to > > > review. > > > > > > I have pinged most threads that are in "Needs review" state and don't > > > apply, compile warning-free, or pass check-world. I'll do some more > > > of that sort of thing, and I'll highlight a different set of patches > > > next week. > > > > Hi Hackers, > > > > Here's a quick status report after the third week, there has been 7 > > entries which were committed in the last week: > > status | 3rd Jan | w1 | w2 | w3 > > -------------------------+-----------+-------+-------+------- > > Needs review: | 177 | 149 | 128 | 118 > > Waiting on Author: | 47 | 60 | 64 | 65 > > Ready for Committer: | 20 | 23 | 26 | 26 > > Committed: | 31 | 40 | 53 | 60 > > Withdrawn: | 4 | 7 | 7 | 8 > > Rejected: | 0 | 0 | 0 | 0 > > Returned with Feedback: | 0 | 0 | 1 | 1 > > Total: | 279 | 279 | 279 | 279 > > > > Here are a few patches which "Needs review", please pick one of these > > and help us in proceeding further: > > 1) Avoid hiding shared filesets in pg_ls_tmpdir (pg_ls_* functions for > > showing metadata ...) | Justin Pryzby > > 2) Fix pg_rewind race condition just after promotion | Heikki Linnakangas > > 3) warn if GUC set to an invalid shared library | Justin Pryzby > > 4) GUC for temporary disabling event triggers | Daniel Gustafsson > > 5) Teach autovacuum.c to launch workers to advance table age without > > attendant antiwraparound cancellation behavior | Peter Geoghegan > > 6) recovery modules | Nathan Bossart > > 7) Add a new pg_walinspect function to extract FPIs from WAL records | > > Bharath Rupireddy > > 8) CI and test improvements | Justin Pryzby > > 9) Test for function error in logrep worker | Anton Melnikov > > 10) Allow tests to pass in OpenSSL FIPS mode | Peter Eisentraut > > 11) Add a test for ldapbindpasswd | Andrew Dunstan, John Naylor > > 12) Add TAP tests for psql \g piped into program | Daniel Vérité > > 13) Support MERGE ... WHEN NOT MATCHED BY SOURCE | Dean Rasheed > > 14) add PROCESS_MAIN to VACUUM | Nathan Bossart > > 15) SQL/JSON | Amit Langote, Nikita Glukhov > > 16) Support MERGE into views | Dean Rasheed > > 17) Exclusion constraints on partitioned tables | Paul Jungwirth > > 18) Ability to reference other extensions by schema in extension > > scripts | Regina Obe > > 19) COPY FROM enable FORCE_NULL/FORCE_NOT_NULL on all columns | Mingli Zhang > > 20) Add support for DEFAULT specification in COPY FROM | Israel Barth > > > > Here is a few patches which are in "Ready for Committer" state, if any > > of the committers has some time to spare, please have a look: > > 1) Transaction timeout | Andrey Borodin > > 2) ANY_VALUE aggregate | Vik Fearing > > 3) POC: Lock updated tuples in tuple_update() and tuple_delete() | > > Alexander Korotkov > > 4) Use fadvise in wal replay | Kirill Reshke, Jakub Wartak > > 5) PG DOCS - pub/sub - specifying optional parameters without values. > > | Peter Smith > > 6) Doc: Rework contrib appendix -- informative titles, tweaked > > sentences | Karl Pinc > > 7) Fix assertion failure with barriers in parallel hash join | Thomas > > Munro, Melanie Plageman > > 8) On client login event trigger | Konstantin Knizhnik, Greg > > Nancarrow, Mikhail Gribkov > > 9) reduce impact of lengthy startup and checkpoint tasks | Nathan Bossart > > 10) Support % wildcard in extension upgrade scripts | Sandro Santilli > > > > If you have submitted a patch and it's in "Waiting for author" state, > > please aim to get it to "Needs review" state soon if you can, as > > that's where people are most likely to be looking for things to > > review. > > > > I have pinged most threads that are in "Needs review" state and don't > > apply, compile warning-free, or pass check-world. I'll do some more > > of that sort of thing, and I'll highlight a different set of patches > > next week. > > Hi, > > Here's a quick status report after the fourth week, with just a few days to > go: > status | 3rd Jan | w1 | w2 | w3 | w4 > -------------------------+-----------+-------+-------+-------+------- > Needs review: | 177 | 149 | 128 | 118 | 112 > Waiting on Author: | 47 | 60 | 64 | 65 | 64 > Ready for Committer: | 20 | 23 | 26 | 26 | 24 > Committed: | 31 | 40 | 53 | 60 | 65 > Withdrawn: | 4 | 7 | 7 | 8 | 12 > Rejected: | 0 | 0 | 0 | 0 | 0 > Returned with Feedback: | 0 | 0 | 1 | 1 | 1 > Total: | 279 | 279 | 279 | 279 | 279 > > There have been 5 patches that were committed in the last week. > I will be updating the patches in the next couple of days before the > commitfest is closed. For patches waiting on author and hasn't had any > updates, I'm planning to mark > as returned with feedback. Anything that is clearly making good > progress but isn't yet ready for committer, I'm going to move to the > next CF. It would be of great help if the patch owner or reviewer can > help in moving the patch in the appropriate direction. Feel free to > change the state if I have updated the patch state wrongly.
Here are the final numbers at the end of the commitfest: status | 3rd Jan | w1 | w2 | w3 | w4 | final -------------------------+-----------+-------+-------+-------+-------+------- Committed: | 31 | 40 | 53 | 60 | 65 | 70 Moved to Next CF | 0 | 0 | 0 | 0 | 0 | 183 Withdrawn: | 4 | 7 | 7 | 8 | 12 | 14 Returned with Feedback: | 0 | 0 | 1 | 1 | 1 | 12 Total: | 279 | 279 | 279 | 279 | 279 | 279 I don't have permissions to close the commitfest, could one of them help in closing the commitfest. Regards, Vignesh