{"push_id":2524588524,"size":7,"distinct_size":1,"ref":"refs/heads/commitfest/18/962","head":"5a35b0fffa1118bbee91e846351d8df2d5501105","before":"0b5384f5c58c32fab581f7b947e3ad8d2bae4d22","commits":[{"sha":"bc19b7836215b1a847524041a1bd138d7bb5cbef","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Minor cleanups for install_llvm_module/uninstall_llvm_module Make macros.\n\nDon't put comments inside the macros, per complaint from Michael Paquier.\n\nQuote target directory path with single quotes, not double; that seems\nto be our project standard.  Not quoting it at all definitely isn't\nper standard.\n\nRemove excess slash in one instance of path.\n\nRemove useless semicolon.\n\nDiscussion: https://postgr.es/m/20180428073935.GB1736@paquier.xyz","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/bc19b7836215b1a847524041a1bd138d7bb5cbef"},{"sha":"f2bb32dbd6af8ec3eb99308289a83b1ff3ca6514","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Un-break contrib install with llvm.\n\nApparently $(foreach ... $(call install_llvm_module,...)) doesn't work\ntoo well without a blank line ending the install_llvm_module macro.\nThe previous coding hackishly dodged this problem with some parens,\nbut that's not really a good solution because make misunderstands\nwhere the command boundaries are that way.\n\nDiscussion: https://postgr.es/m/20180428073935.GB1736@paquier.xyz","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/f2bb32dbd6af8ec3eb99308289a83b1ff3ca6514"},{"sha":"45c6d75f8cd2399181063b4e0ec11c2664ebeda6","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Clarify handling of special-case values in bootstrap catalog data.\n\nI (tgl) originally coded the special case for pg_proc.pronargs as\nthough it were a kind of default value.  It seems better though to\ntreat computable columns as an independent concern: this makes the\ncode clearer, and probably a bit faster too since we needn't do\nwork inside the per-column loop.\n\nImprove related comments, as well, in the expectation that there\nmight be more cases like this in future.\n\nJohn Naylor, some additional comment-hacking by me\n\nDiscussion: https://postgr.es/m/CAJVSVGW-D7OobzU=dybVT2JqZAx-4X1yvBJdavBmqQL05Q6CLw@mail.gmail.com","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/45c6d75f8cd2399181063b4e0ec11c2664ebeda6"},{"sha":"84549ebd4c4d1941006056cfb2455a4983fb15a2","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Tweak reformat_dat_file.pl to make it more easily hand-invokable.\n\nUse the same code we already applied in duplicate_oids and unused_oids\nto let this script find Catalog.pm without help.  This removes the need\nto supply a -I switch in most cases.\n\nAlso, mark the script executable, again to follow the precedent of\nduplicate_oids and unused_oids.  Now you can just do\n\"./reformat_dat_file.pl pg_proc.dat\"\nif you want to reformat only one or a few .dat files rather than all.\n\nIt'd be possible to remove the -I switches in the Makefile's convenience\ntargets, but I chose to leave them: they don't hurt anything, and it's\npossible that in weird VPATH situations they might be of value.","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/84549ebd4c4d1941006056cfb2455a4983fb15a2"},{"sha":"cfffe83ba82021a1819a656e7ec5c28fb3a99152","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Fix incorrect field type for PlannedStmt.jitFlags in outfuncs/readfuncs.\n\nThis field was a bool at one point, but now it's an int.\nSpotted by Hari Babu; trivial patch is by Ashutosh Bapat.\n\nDiscussion: https://postgr.es/m/CAJrrPGedKiFE2fqntSauUfhapCksOJzam+QtHfSgx86LhXLeOQ@mail.gmail.com","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/cfffe83ba82021a1819a656e7ec5c28fb3a99152"},{"sha":"9cb7db3f0c1f554cdcbbd97f78a119a19756e6ef","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"In AtEOXact_Files, complain if any files remain unclosed at commit.\n\nThis change makes this module act more like most of our other low-level\nresource management modules.  It's a caller error if something is not\nexplicitly closed by the end of a successful transaction, so issue\na WARNING about it.  This would not actually have caught the file leak\nbug fixed in commit 231bcd080, because that was in a transaction-abort\npath; but it still seems like a good, and pretty cheap, cross-check.\n\nDiscussion: https://postgr.es/m/152056616579.4966.583293218357089052@wrigleys.postgresql.org","distinct":false,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/9cb7db3f0c1f554cdcbbd97f78a119a19756e6ef"},{"sha":"5a35b0fffa1118bbee91e846351d8df2d5501105","author":{"name":"Commitfest Bot","email":"b9e638e07cc2e2abf56c9c0cad7954f2e3bc1641@cputube.org"},"message":"[CF 18/962] new plpgsql extra_checks\n\nThis commit was automatically generated by cfbot at commitfest.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 or the master branch changes.\n\nCommitfest entry: https://commitfest.postgresql.org/18/962\nPatch(es): https://www.postgresql.org/message-id/CAFj8pRBFHUHxRz%3DaYt_L7L%2B8gqyOJT3SAkpath8SvZCQ0uj2NA%40mail.gmail.com\nAuthor(s): Pavel Stehule","distinct":true,"url":"https://api.github.com/repos/postgresql-cfbot/postgresql/commits/5a35b0fffa1118bbee91e846351d8df2d5501105"}]}
{"actor":{"display_login":"postgresql-cfbot"}}