{"push_id":1062051649,"size":5,"distinct_size":5,"ref":"refs/heads/dxldmlsentinel","head":"63a3ce808b3a61279629484fab690958a2ea7cf0","before":"871805033fa702061a6d316d037b97780ba83924","commits":[{"sha":"d94966700d2fa2e46b0410f7b9140a5617e2b87d","author":{"email":"57813d5da8796a750bb6b13c5838fc4a0a43c338@users.noreply.github.com","name":"Kuien Liu"},"message":"Support the aggregation transition datatype to be \"internal\" (#588)\n\n* Support the aggregation transition datatype to be \"internal\"\r\n\r\nBackport below commits from upstream:\r\n\r\n    commit 22d9ddbb35f5d316312a2220041ff17744a72330\r\n    Author: Tom Lane <tgl@sss.pgh.pa.us>\r\n    Date:   Sat Aug 14 15:47:13 2010 +0000\r\n\r\n        Fix planner to make a reasonable assumption about the amount of memory space\r\n        used by array_agg(), string_agg(), and similar aggregate functions that use\r\n        \"internal\" as their transition datatype.  The previous coding thought this\r\n        took *no* extra space, since \"internal\" is pass-by-value; but actually these\r\n        aggregates typically consume a great deal of space.  Per bug #5608 from\r\n        Itagaki Takahiro, and fix suggestion from Hitoshi Harada.\r\n\r\n        Back-patch to 8.4, where array_agg was introduced.\r\n\r\n    commit 9e0247aba5e2cac1542823a194d4dc2ea1c3abb6\r\n    Author: Tom Lane <tgl@sss.pgh.pa.us>\r\n    Date:   Fri Nov 14 19:47:50 2008 +0000\r\n\r\n        In CREATE AGGREGATE, allow the transition datatype to be \"internal\", but only\r\n        if the user is superuser.  This makes available to extension modules the same\r\n        sort of trick being practiced by array_agg().  The reason for the superuser\r\n        restriction is that you could crash the system by connecting up an\r\n        incompatible pair of internal-using functions as an aggregate.  It shouldn't\r\n        interfere with any legitimate use, since you'd have to be superuser to create\r\n        the internal-using transition and final functions anyway.\r\n\r\nBesides, we cannot find regression tests in above commits, so we\r\nadd a set of tests for this feature, a part of them come from\r\nlegacy extension \"intagg\" regression tests.\r\n\r\nConflicts:\r\n\tsrc/test/regress/expected/arrays.out\r\n\tsrc/test/regress/sql/arrays.sql\r\n\r\n* add tests to #588: run with role of (non-)superuser\r\n\r\n    #588: Support the aggregate's transype to be \"internal\"\r\n\r\nNormally, the aggregate's transtype can't be a pseudo-type.\r\nHowever, we can allow \"internal\" for functions that want to\r\npass pointers to private data structures; but allow that only\r\nto superusers. Here we add tests to verify this constraint.","distinct":true,"url":"https://api.github.com/repos/danielgustafsson/gpdb/commits/d94966700d2fa2e46b0410f7b9140a5617e2b87d"},{"sha":"9a14432338cc5e0d112599139f684682cdb58d90","author":{"email":"453fd3aa6d9e84cafb9bceac21e8513230792eac@pivotal.io","name":"Kenan Yao"},"message":"Wrong data inserted when COPY files to an altered partition table.","distinct":true,"url":"https://api.github.com/repos/danielgustafsson/gpdb/commits/9a14432338cc5e0d112599139f684682cdb58d90"},{"sha":"84f2c1e3bafad60447894b9dff69c41cf1146fc7","author":{"email":"9b5ecfe15c9cd044f4ab483a287130e4bc266a44@pivotal.io","name":"Jimmy Yih"},"message":"Change gpconfig exit status for option parsing.\n\nCurrently, the gpconfig utility is exiting with status 0 when one of\nthe option parsing checks fail.  It should return 1 instead.","distinct":true,"url":"https://api.github.com/repos/danielgustafsson/gpdb/commits/84f2c1e3bafad60447894b9dff69c41cf1146fc7"},{"sha":"5cdbbef43dc1112c35d1db07f35779f67263c6cf","author":{"email":"53f38ff1dce97d19d07cddf5d958aad389b7c613@pivotal.io","name":"Abhijit Subramanya"},"message":"Make the error log tests for external tables more deterministic.","distinct":true,"url":"https://api.github.com/repos/danielgustafsson/gpdb/commits/5cdbbef43dc1112c35d1db07f35779f67263c6cf"},{"sha":"63a3ce808b3a61279629484fab690958a2ea7cf0","author":{"email":"c8568db482997e400a30352c275ad00bc4c4e5db@pivotal.io","name":"Daniel Gustafsson"},"message":"Handle the Sentinel case in node creation\n\nWe really shouldn't get a sentinel node here but if we for some reason\ndo we need/should handle it by erroring out.","distinct":true,"url":"https://api.github.com/repos/danielgustafsson/gpdb/commits/63a3ce808b3a61279629484fab690958a2ea7cf0"}]}