payload: {"push_id":4247238238,"size":1000,"distinct_size":2199,"ref":"refs/heads/dev/clang-fallthrough","head":"2d3c87780fe2923136c51d4da9c888b6dd6a737c","before":"7538a38d23b57c0327fce3e7b4441f623f4155da","commits":[{"sha":"e66197fa2efa8ae0cab1eed6b2257ab4e2134b1e","author":{"name":"Andres Freund","email":"883768b6dd2c42aea0031b24be8a2da40fef4b64@anarazel.de"},"message":"plperl: Correctly handle empty arrays in plperl_ref_from_pg_array.\n\nplperl_ref_from_pg_array() didn't consider the case that postgrs arrays\ncan have 0 dimensions (when they're empty) and accessed the first\ndimension without a check. Fix that by special casing the empty array\ncase.\n\nAuthor: Alex Hunsaker\nReported-By: Andres Freund / valgrind / buildfarm animal skink\nDiscussion: 20160308063240.usnzg6bsbjrne667@alap3.anarazel.de\nBackpatch: 9.1-","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/e66197fa2efa8ae0cab1eed6b2257ab4e2134b1e"},{"sha":"61fd218930db53079e5f001dd4ea2fd53afd1b95","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Fix minor thinko in pathification code.\n\nI passed the wrong \"root\" struct to create_pathtarget in build_minmax_path.\nSince the subroot is a clone of the outer root, this would not cause any\nserious problems, but it would waste some cycles because\nset_pathtarget_cost_width would not have access to Var width estimates\nset up while running query_planner on the subroot.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/61fd218930db53079e5f001dd4ea2fd53afd1b95"},{"sha":"7a1d4a2448c34ed4669d67ae4f24c594545f10b5","author":{"name":"Andres Freund","email":"883768b6dd2c42aea0031b24be8a2da40fef4b64@anarazel.de"},"message":"ltree: Zero padding bytes when allocating memory for externally visible data.\n\nltree/ltree_gist/ltxtquery's headers stores data at MAXALIGN alignment,\nrequiring some padding bytes. So far we left these uninitialized. Zero\nthose by using palloc0.\n\nAuthor: Andres Freund\nReported-By: Andres Freund / valgrind / buildarm animal skink\nBackpatch: 9.1-","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/7a1d4a2448c34ed4669d67ae4f24c594545f10b5"},{"sha":"0d0644dce82a87ef4b670fb60709ef1fd6a3030c","author":{"name":"Peter Eisentraut","email":"c50e6bf7013fce877ffb25104d778a36de31e297@gmx.net"},"message":"ecpg: Fix typo\n\nGCC 6 points out the redundant conditions, which were apparently typos.\n\nReviewed-by: Thomas Munro <thomas.munro@enterprisedb.com>","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/0d0644dce82a87ef4b670fb60709ef1fd6a3030c"},{"sha":"92d4294d4bbddf94cec0dce729ade7bb2aada1b7","author":{"name":"Peter Eisentraut","email":"c50e6bf7013fce877ffb25104d778a36de31e297@gmx.net"},"message":"psql: Fix some strange code in SQL help creation\n\nStruct QL_HELP used to be defined as static in the sql_help.h header\nfile, which is included in sql_help.c and help.c, thus creating two\nseparate instances of the struct.  This causes a warning from GCC 6,\nbecause the struct is not used in sql_help.c.\n\nInstead, declare the struct as extern in the header file and define it\nin sql_help.c.  This also allows making a bunch of functions static\nbecause they are no longer needed outside of sql_help.c.\n\nReviewed-by: Thomas Munro <thomas.munro@enterprisedb.com>","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/92d4294d4bbddf94cec0dce729ade7bb2aada1b7"},{"sha":"a2fd62dd53fb606dee69e0f4eb12289c87f5c8b1","author":{"name":"Peter Eisentraut","email":"c50e6bf7013fce877ffb25104d778a36de31e297@gmx.net"},"message":"Suppress GCC 6 warning about self-comparison\n\nReviewed-by: Thomas Munro <thomas.munro@enterprisedb.com>","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/a2fd62dd53fb606dee69e0f4eb12289c87f5c8b1"},{"sha":"a40814d7aacbda9266c703cc011d3506668899de","author":{"name":"Peter Eisentraut","email":"c50e6bf7013fce877ffb25104d778a36de31e297@gmx.net"},"message":"Handle invalid libpq sockets in more places\n\nAlso, make error messages consistent.\n\nFrom: Michael Paquier <michael.paquier@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/a40814d7aacbda9266c703cc011d3506668899de"},{"sha":"9e8b99420fe5f80495ada8dc50aeb7b954b33093","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Improve handling of group-column indexes in GroupingSetsPath.\n\nInstead of having planner.c compute a groupColIdx array and store it in\nGroupingSetsPaths, make create_groupingsets_plan() find the grouping\ncolumns by searching in the child plan node's tlist.  Although that's\nprobably a bit slower for create_groupingsets_plan(), it's more like\nthe way every other plan node type does this, and it provides positive\nconfirmation that we know which child output columns we're supposed to be\ngrouping on.  (Indeed, looking at this now, I'm not at all sure that it\nwasn't broken before, because create_groupingsets_plan() isn't demanding\nan exact tlist match from its child node.)  Also, this allows substantial\nsimplification in planner.c, because it no longer needs to compute the\ngroupColIdx array at all; no other cases were using it.\n\nI'd intended to put off this refactoring until later (like 9.7), but\nin view of the likely bug fix and the need to rationalize planner.c's\ntlist handling so we can do something sane with Konstantin Knizhnik's\nfunction-evaluation-postponement patch, I think it can't wait.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/9e8b99420fe5f80495ada8dc50aeb7b954b33093"},{"sha":"5e43bee8307f1f6f87894c9a4bd9f9045f45c064","author":{"name":"Andres Freund","email":"883768b6dd2c42aea0031b24be8a2da40fef4b64@anarazel.de"},"message":"Add valgrind suppressions for bootstrap related code.\n\nAuthor: Andres Freund\nBackpatch: 9.4, where we started to maintain valgrind suppressions","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/5e43bee8307f1f6f87894c9a4bd9f9045f45c064"},{"sha":"2f1f4439306d2793492e49366d5911e48aa2c4b1","author":{"name":"Andres Freund","email":"883768b6dd2c42aea0031b24be8a2da40fef4b64@anarazel.de"},"message":"Add valgrind suppressions for python code.\n\nPython's allocator does some low-level tricks for efficiency;\nunfortunately they trigger valgrind errors. Those tricks can be disabled\nmaking instrumentation easier; but few people testing postgres will have\nsuch a build of python. So add broad suppressions of the resulting\nerrors.\n\nSee also https://svn.python.org/projects/python/trunk/Misc/README.valgrind\n\nThis possibly will suppress valid errors, but without it it's basically\nimpossible to use valgrind with plpython code.\n\nAuthor: Andres Freund\nBackpatch: 9.4, where we started to maintain valgrind suppressions","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/2f1f4439306d2793492e49366d5911e48aa2c4b1"},{"sha":"51c0f63e4d76a86b44e87876a6addcfffb01ec28","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Improve handling of pathtargets in planner.c.\n\nRefactor so that the internal APIs in planner.c deal in PathTargets not\ntargetlists, and establish a more regular structure for deriving the\ntargets needed for successive steps.\n\nThere is more that could be done here; calculating the eval costs of each\nsuccessive target independently is both inefficient and wrong in detail,\nsince we won't actually recompute values available from the input node's\ntlist.  But it's no worse than what happened before the pathification\nrewrite.  In any case this seems like a good starting point for considering\nhow to handle Konstantin Knizhnik's function-evaluation-postponement patch.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/51c0f63e4d76a86b44e87876a6addcfffb01ec28"},{"sha":"d31f20e2b5a246f276c73134b610ac7a2f34e274","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Fix copy-and-pasteo in comment.\n\nWensheng Zhang","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/d31f20e2b5a246f276c73134b610ac7a2f34e274"},{"sha":"aa09cd242fa7e3a694a31f8aed521e80d1e626a4","author":{"name":"Robert Haas","email":"cc4ab156f0be9feefe10b35a552fd319cd7679ae@postgresql.org"},"message":"postgres_fdw: Consider foreign joining and foreign sorting together.\n\nCommit ccd8f97922944566d26c7d90eb67ab7848ee9905 gave us the ability to\nrequest that the remote side sort the data, and, later, commit\ne4106b2528727c4b48639c0e12bf2f70a766b910 gave us the ability to\nrequest that the remote side perform the join for us rather than doing\nit locally.  But we could not do both things at the same time: a\nremote SQL query that had an ORDER BY clause would never be a join.\nThis commit adds that capability.\n\nAshutosh Bapat, reviewed by me.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/aa09cd242fa7e3a694a31f8aed521e80d1e626a4"},{"sha":"8776c15c85322612b9bf79daf50f74be71c12e05","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Fix incorrect tlist generation in create_gather_plan().\n\nThis function is written as though Gather doesn't project; but it does.\nEven if it did not project, though, we must use build_path_tlist to ensure\nthat the output columns receive correct sortgroupref labeling.\n\nPer report from Amit Kapila.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/8776c15c85322612b9bf79daf50f74be71c12e05"},{"sha":"b6fb6471f6afaf649e52f38269fd8c5c60647669","author":{"name":"Robert Haas","email":"cc4ab156f0be9feefe10b35a552fd319cd7679ae@postgresql.org"},"message":"Add a generic command progress reporting facility.\n\nUsing this facility, any utility command can report the target relation\nupon which it is operating, if there is one, and up to 10 64-bit\ncounters; the intent of this is that users should be able to figure out\nwhat a utility command is doing without having to resort to ugly hacks\nlike attaching strace to a backend.\n\nAs a demonstration, this adds very crude reporting to lazy vacuum; we\njust report the target relation and nothing else.  A forthcoming patch\nwill make VACUUM report a bunch of additional data that will make this\nmuch more interesting.  But this gets the basic framework in place.\n\nVinayak Pokale, Rahila Syed, Amit Langote, Robert Haas, reviewed by\nKyotaro Horiguchi, Jim Nasby, Thom Brown, Masahiko Sawada, Fujii Masao,\nand Masanori Oyama.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/b6fb6471f6afaf649e52f38269fd8c5c60647669"},{"sha":"188f359d39ed65b5f3ddc1f397140fb9d153e61a","author":{"name":"Alvaro Herrera","email":"3c84f0de6f8a0f296d36cdd9f57fa166641c96b1@alvh.no-ip.org"},"message":"pgcrypto: support changing S2K iteration count\n\npgcrypto already supports key-stretching during symmetric encryption,\nincluding the salted-and-iterated method; but the number of iterations\nwas not configurable.  This commit implements a new s2k-count parameter\nto pgp_sym_encrypt() which permits selecting a larger number of\niterations.\n\nAuthor: Jeff Janes","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/188f359d39ed65b5f3ddc1f397140fb9d153e61a"},{"sha":"accf7616ff2854640523d5e42ffa5420cc5a3b51","author":{"name":"Robert Haas","email":"cc4ab156f0be9feefe10b35a552fd319cd7679ae@postgresql.org"},"message":"pgbench: When -T is used, don't wait for transactions beyond end of run.\n\nAt low rates, this can lead to pgbench taking significantly longer to\nterminate than the user might expect.  Repair.\n\nFabien Coelho, reviewed by Aleksander Alekseev, Álvaro Herrera, and me.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/accf7616ff2854640523d5e42ffa5420cc5a3b51"},{"sha":"be060cbcd42737693f6fd425db4c139121181cce","author":{"name":"Robert Haas","email":"cc4ab156f0be9feefe10b35a552fd319cd7679ae@postgresql.org"},"message":"Re-pgindent vacuumlazy.c.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/be060cbcd42737693f6fd425db4c139121181cce"},{"sha":"a298a1e06fb0574c898a07761f9f86c2a323919e","author":{"name":"Tom Lane","email":"ffeaa947aef224da9ba99b6a18bdd74288df0e3e@sss.pgh.pa.us"},"message":"Fix incorrect handling of NULL index entries in indexed ROW() comparisons.\n\nAn index search using a row comparison such as ROW(a, b) > ROW('x', 'y')\nwould stop upon reaching a NULL entry in the \"b\" column, ignoring the\nfact that there might be non-NULL \"b\" values associated with later values\nof \"a\".  This happens because _bt_mark_scankey_required() marks the\nsubsidiary scankey for \"b\" as required, which is just wrong: it's for\na column after the one with the first inequality key (namely \"a\"), and\nthus can't be considered a required match.\n\nThis bit of brain fade dates back to the very beginnings of our support\nfor indexed ROW() comparisons, in 2006.  Kind of astonishing that no one\ncame across it before Glen Takahashi, in bug #14010.\n\nBack-patch to all supported versions.\n\nNote: the given test case doesn't actually fail in unpatched 9.1, evidently\nbecause the fix for bug #6278 (i.e., stopping at nulls in either scan\ndirection) is required to make it fail.  I'm sure I could devise a case\nthat fails in 9.1 as well, perhaps with something involving making a cursor\nback up; but it doesn't seem worth the trouble.","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/a298a1e06fb0574c898a07761f9f86c2a323919e"},{"sha":"a31aaec40643e0074f30b1683953cb0b3ea27036","author":{"name":"Alvaro Herrera","email":"3c84f0de6f8a0f296d36cdd9f57fa166641c96b1@alvh.no-ip.org"},"message":"Add filter capability to RecursiveCopy::copypath\n\nThis allows skipping copying certain files and subdirectories in tests.\nThis is useful in some circumstances such as copying a data directory;\nfuture tests want this feature.\n\nAlso POD-ify the module.\n\nAuthors: Craig Ringer, Pallavi Sontakke\nReviewed-By: Álvaro Herrera","distinct":true,"url":"https://api.github.com/repos/jchampio/gpdb/commits/a31aaec40643e0074f30b1683953cb0b3ea27036"}]}
other: {"actor":{"display_login":"jchampio"}}