> transformation. For these reasons, variant A where we put the > LimitOption between the two int8 expression nodes feels like the > "okay" approach here. But we must document this expectation in the > structure, and check for more grammar variants of LIMIT and OFFSET > clauses in pgss.
Please see the attached. Variant A with comments and some additional test cases. It should be noted that we currently have "WITH TIES/ROWS ONLY" tests in pg_s_s, so I added another case to show "FETCH FIRST 2 ROW ONLY" and "LIMIT 2" are the same queryId and also added a query that uses both a LIMIT and OFFSET. I could not think of other cases we need to cover. -- Sami
v1-0001-Fix-QueryId-collision-for-LIMIT-and-OFFSET.patch
Description: Binary data