In [1], Yaoguang reported an Assert failure in expand_grouping_sets. Since beta2 deadline is looming, I pushed a quick fix for that.
As mentioned over on bugs, only 1 test triggers that code and because the List of IntLists always had an empty list as the first element due to the code just above sorting the top-level List by the number of elements each of the contained IntLists, the NIL was always at the start of the top-level List. It wasn't too hard to modify the test to change that. I wonder if the testing for the feature is just a bit too light. Would it maybe be worth adding a GROUP BY DISTINCT with GROUPING SETS test? Any thoughts? David [1] https://www.postgresql.org/message-id/17067-665d50fa321f7...@postgresql.org
