{"push_id":3748579297,"size":12,"distinct_size":1,"ref":"refs/heads/commitfest/23/2094","head":"658b65e9dc1c3321b4bbf75acf68ed830dd173b6","before":"941c7d0cb2f183e49f141499e4649e72182d0dd0","commits":[{"sha":"12e037e2090b1a6446376814ea0419463b02c254","author":{"name":"Peter Eisentraut","email":"4b8373d016f277527198385ba72fda0feb5da015@eisentraut.org"},"message":"Upgrade internal error message to external\n\nAs part of REINDEX CONCURRENTLY, this formerly internal-only error\nmessage becomes potentially user-visible (see regression tests), so\nchange from errmsg_internal() to errmsg(), and update comment.","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/12e037e2090b1a6446376814ea0419463b02c254"},{"sha":"f31111bbe81db0e84fb486c6423a234c47091b30","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Drop test user when done with it.\n\nCommit d7f8d26d9 added a test case that created a user, but forgot\nto drop it again.  This is no good; for one thing, it causes repeated\n\"make installcheck\" runs to fail.","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/f31111bbe81db0e84fb486c6423a234c47091b30"},{"sha":"b00326df7ad4ab7d21aad3b6fe6558ea6b61897d","author":{"name":"Peter Geoghegan","email":"96dd8d96902e9c5af3b96b1eaa9c80cbc615d290@bowt.ie"},"message":"Correct obsolete amcheck comments.\n\nOversight in commit dd299df8.","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/b00326df7ad4ab7d21aad3b6fe6558ea6b61897d"},{"sha":"f946a409143d01951411382fbc3c91c7eb640094","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Further fix ALTER COLUMN TYPE's handling of indexes and index constraints.\n\nThis patch reverts all the code changes of commit e76de8861, which turns\nout to have been seriously misguided.  We can't wait till later to compute\nthe definition string for an index; we must capture that before applying\nthe data type change for any column it depends on, else ruleutils.c will\ndeliverr wrong/misleading results.  (This fine point was documented\nnowhere, of course.)\n\nI'd also managed to forget that ATExecAlterColumnType executes once per\nALTER COLUMN TYPE clause, not once per statement; which resulted in the\ncode being basically completely broken for any case in which multiple ALTER\nCOLUMN TYPE clauses are applied to a table having non-constraint indexes\nthat must be rebuilt.  Through very bad luck, none of the existing test\ncases nor the ones added by e76de8861 caught that, but of course it was\nsoon found in the field.\n\nThe previous patch also had an implicit assumption that if a constraint's\nindex had a dependency on a table column, so would the constraint --- but\nthat isn't actually true, so it didn't fix such cases.\n\nInstead of trying to delete unneeded index dependencies later, do the\nis-there-a-constraint lookup immediately on seeing an index dependency,\nand switch to remembering the constraint if so.  In the unusual case of\nmultiple column dependencies for a constraint index, this will result in\nduplicate constraint lookups, but that's not that horrible compared to all\nthe other work that happens here.  Besides, such cases did not work at all\nbefore, so it's hard to argue that they're performance-critical for anyone.\n\nPer bug #15865 from Keith Fiske.  As before, back-patch to all supported\nbranches.\n\nDiscussion: https://postgr.es/m/15865-17940eacc8f8b081@postgresql.org","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/f946a409143d01951411382fbc3c91c7eb640094"},{"sha":"eb8d05bfeca416a970cc33af5915ec7b4f4a2f69","author":{"name":"Peter Eisentraut","email":"4b8373d016f277527198385ba72fda0feb5da015@eisentraut.org"},"message":"Correct script name in README file","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/eb8d05bfeca416a970cc33af5915ec7b4f4a2f69"},{"sha":"2cadefbb97ff8e5566476780dc200db06af6ba09","author":{"name":"Peter Eisentraut","email":"4b8373d016f277527198385ba72fda0feb5da015@eisentraut.org"},"message":"Make script output more pgindent compatible","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/2cadefbb97ff8e5566476780dc200db06af6ba09"},{"sha":"82be666ee36842af2a1b81e570bd5578f264de65","author":{"name":"Peter Eisentraut","email":"4b8373d016f277527198385ba72fda0feb5da015@eisentraut.org"},"message":"Update unicode_norm_table.h to Unicode 12.1.0","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/82be666ee36842af2a1b81e570bd5578f264de65"},{"sha":"ccfcc8fdbd9bdbfd18fda5d7c698af8d175f5319","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Purely-cosmetic adjustments in tablecmds.c.\n\nMove ATExecAlterColumnGenericOptions away from where it was unthinkingly\ndropped, in the middle of a lot of ALTER COLUMN TYPE code.  I don't have\nany high principles about where to put it instead, so let's just put it\nafter ALTER COLUMN TYPE and before ALTER OWNER, matching existing\ndecisions about how to order related code stanzas.\n\nAlso add the minimal function header comment that the original author\nwas too cool to bother with.\n\nAlong the way, upgrade header comments for nearby ALTER COLUMN TYPE\nfunctions.\n\nDiscussion: https://postgr.es/m/14787.1561403130@sss.pgh.pa.us","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/ccfcc8fdbd9bdbfd18fda5d7c698af8d175f5319"},{"sha":"a2dec374807565f6337915fb236bb3ae37af0644","author":{"name":"Thomas Munro","email":"1ff054ef0838bf068a4b7c40eef026e560a37025@postgresql.org"},"message":"Remove misleading comment from pathnodes.h.\n\nAs of commit e5253fdc, it is no longer true that the leader always\nexecutes the subplan of a Gather Merge node.  Remove comment to that\neffect.\n\nBack-patch to 11.\n\nDiscussion: https://postgr.es/m/CA%2BhUKGJEaZJYezXAOutuiWT%2BfxCA44%2BoKtVPAND2ubLiigR%3D-w%40mail.gmail.com","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/a2dec374807565f6337915fb236bb3ae37af0644"},{"sha":"0089c3059cb6b3dd20cf072f26d1a7a33412df39","author":{"name":"Thomas Munro","email":"1ff054ef0838bf068a4b7c40eef026e560a37025@postgresql.org"},"message":"Don't unset MAKEFLAGS in non-GNU Makefile.\n\nIt's useful to be able to pass down options like -s and -j.\n\nBack-patch to 9.5, like commit a76200de.\n\nDiscussion: https://postgr.es/m/CA%2BhUKG%2Be1M8-BbL%3DPqhTp6oO6XPO6%2Bs9WGQMLfbuZ%3DG9CtzyXg%40mail.gmail.com","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/0089c3059cb6b3dd20cf072f26d1a7a33412df39"},{"sha":"ce59b75d449d9152667ce3e9eab33ef2872bfd98","author":{"name":"Michael Paquier","email":"17b9e1c64588c7fa6419b4d29dc1f4426279ba01@paquier.xyz"},"message":"Add toast-level reloption for vacuum_index_cleanup\n\na96c41f has introduced the option for heap, but it still lacked the\nvariant to control the behavior for toast relations.\n\nWhile on it, refactor the tests so as they stress more scenarios with\nthe various values that vacuum_index_cleanup can use.  It would be\nuseful to couple those tests with pageinspect to check that pages are\nactually cleaned up, but this is left for later.\n\nAuthor: Masahiko Sawada, Michael Paquier\nReviewed-by: Peter Geoghegan\nDiscussion: https://postgr.es/m/CAD21AoCqs8iN04RX=i1KtLSaX5RrTEM04b7NHYps4+rqtpWNEg@mail.gmail.com","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/ce59b75d449d9152667ce3e9eab33ef2872bfd98"},{"sha":"658b65e9dc1c3321b4bbf75acf68ed830dd173b6","author":{"name":"Commitfest Bot","email":"b9e638e07cc2e2abf56c9c0cad7954f2e3bc1641@cputube.org"},"message":"[CF 23/2094] libpq - improve PQexec documentation\n\nThis commit was automatically generated by a robot at cfbot.cputube.org.\nIt is based on patches submitted to the PostgreSQL mailing lists and\nregistered in the PostgreSQL Commitfest application.\n\nThis branch will be overwritten each time a new patch version is posted to\nthe email thread, and also periodically to check for bitrot caused by changes\non the master branch.\n\nCommitfest entry: https://commitfest.postgresql.org/23/2094\nPatch(es): https://www.postgresql.org/message-id/alpine.DEB.2.21.1904121016310.8912@lancre\nAuthor(s): Fabien Coelho","distinct":true,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/658b65e9dc1c3321b4bbf75acf68ed830dd173b6"}]}
{"actor":{"display_login":"postgresql-cfbot"}}