{"push_id":1297541059,"size":5,"distinct_size":0,"ref":"refs/heads/master","head":"55c3391d1e6a201b5b891781d21fe682a8c64fe6","before":"9f478b4f19d8e26300ae19e42c26343f5791e32a","commits":[{"sha":"0dba54f1666ead71c54ce100b39efda67596d297","author":{"name":"Andres Freund","email":"c6cbfe878b4ee67646e7630734db67f536518d54@anarazel.de"},"message":"Remove user_relns() SRF from regression tests.\n\nThe output of the function changes whenever previous (or, as in this\ncase, concurrent) tests leave a table in place. That causes unneeded\nchurn.\n\nThis should fix failures due to the tests added bfe16d1a5, like on\nlapwing, caused by the tsrf test running concurrently with misc. Those\ncould also have been addressed by using temp tables, but that test has\nannoyed me before.\n\nDiscussion: <27626.1473729905@sss.pgh.pa.us>","distinct":false,"url":"https://api.github.com/repos/michaelpq/postgres/commits/0dba54f1666ead71c54ce100b39efda67596d297"},{"sha":"445a38aba26cb80a4506af2248e3b425f795a099","author":{"name":"Robert Haas","email":"93159bdbbeed5b47be970d58edfbdf1c3ce55e4c@postgresql.org"},"message":"Have heapam.h include lockdefs.h rather than lock.h.\n\nlockdefs.h was only split from lock.h relatively recently, and\nrepresents a minimal subset of the old lock.h.  heapam.h only needs\nthat smaller subset, so adjust it to include only that.  This requires\nsome corresponding adjustments elsewhere.\n\nPeter Geoghegan","distinct":false,"url":"https://api.github.com/repos/michaelpq/postgres/commits/445a38aba26cb80a4506af2248e3b425f795a099"},{"sha":"a4c35ea1c2f05dd5b56739fcd0dc36a4870ea0c0","author":{"name":"Tom Lane","email":"9ca635a44fdf5aa326758d8d245579b9ae84653c@sss.pgh.pa.us"},"message":"Improve parser's and planner's handling of set-returning functions.\n\nTeach the parser to reject misplaced set-returning functions during parse\nanalysis using p_expr_kind, in much the same way as we do for aggregates\nand window functions (cf commit eaccfded9).  While this isn't complete\n(it misses nesting-based restrictions), it's much better than the previous\nerror reporting for such cases, and it allows elimination of assorted\nad-hoc expression_returns_set() error checks.  We could add nesting checks\nlater if it seems important to catch all cases at parse time.\n\nThere is one case the parser will now throw error for although previous\nversions allowed it, which is SRFs in the tlist of an UPDATE.  That never\nbehaved sensibly (since it's ill-defined which generated row should be\nused to perform the update) and it's hard to see why it should not be\ntreated as an error.  It's a release-note-worthy change though.\n\nAlso, add a new Query field hasTargetSRFs reporting whether there are\nany SRFs in the targetlist (including GROUP BY/ORDER BY expressions).\nThe parser can now set that basically for free during parse analysis,\nand we can use it in a number of places to avoid expression_returns_set\nsearches.  (There will be more such checks soon.)  In some places, this\nallows decontorting the logic since it's no longer expensive to check for\nSRFs in the tlist --- so I made the checks parallel to the handling of\nhasAggs/hasWindowFuncs wherever it seemed appropriate.\n\ncatversion bump because adding a Query field changes stored rules.\n\nAndres Freund and Tom Lane\n\nDiscussion: <24639.1473782855@sss.pgh.pa.us>","distinct":false,"url":"https://api.github.com/repos/michaelpq/postgres/commits/a4c35ea1c2f05dd5b56739fcd0dc36a4870ea0c0"},{"sha":"fdc79e1909dc3866a385ffb74bdd6ce6a082a300","author":{"name":"Tom Lane","email":"9ca635a44fdf5aa326758d8d245579b9ae84653c@sss.pgh.pa.us"},"message":"Fix executor/README to reflect disallowing SRFs in UPDATE.\n\nThe parenthetical comment here is obsoleted by commit a4c35ea1c.\nNoted by Andres Freund.","distinct":false,"url":"https://api.github.com/repos/michaelpq/postgres/commits/fdc79e1909dc3866a385ffb74bdd6ce6a082a300"},{"sha":"55c3391d1e6a201b5b891781d21fe682a8c64fe6","author":{"name":"Tom Lane","email":"9ca635a44fdf5aa326758d8d245579b9ae84653c@sss.pgh.pa.us"},"message":"Be pickier about converting between Name and Datum.\n\nWe were misapplying NameGetDatum() to plain C strings in some places.\nThis worked, because it was just a pointer cast anyway, but it's a type\ncheat in some sense.  Use CStringGetDatum instead, and modify the\nNameGetDatum macro so it won't compile if applied to something that's\nnot a pointer to NameData.  This should result in no changes to\ngenerated code, but it is logically cleaner.\n\nMark Dilger, tweaked a bit by me\n\nDiscussion: <EFD8AC94-4C1F-40C1-A5EA-304080089C1B@gmail.com>","distinct":false,"url":"https://api.github.com/repos/michaelpq/postgres/commits/55c3391d1e6a201b5b891781d21fe682a8c64fe6"}]}
{"actor":{"display_login":"michaelpq"}}