sql-parser icon indicating copy to clipboard operation
sql-parser copied to clipboard

Support standard UNION/INTERSECT syntax

Open dinodork opened this issue 1 year ago • 9 comments

The SQL standard allows parentheses around all query expressions (SELECTs) in a UNION. The Bison parser cannot currently handle queries like (SELECT 1) UNION (SELECT 1), let alone ((SELECT 1) UNION (SELECT 1)).

dinodork avatar Aug 30 '24 10:08 dinodork

Just approved to run the actions, I will review your PR in the next week. I noticed that the gcc-6 stage fails because manually cloning does not work with forked repositories, I will see if I can provide a fix for that.

dey4ss avatar Sep 03 '24 06:09 dey4ss

Just approved to run the actions, I will review your PR in the next week. I noticed that the gcc-6 stage fails because manually cloning does not work with forked repositories, I will see if I can provide a fix for that.

Thanks a lot. Since you're on it, I removed to last commit where I was trying to fix that issue, not doing a very good job at it.

dinodork avatar Sep 03 '24 08:09 dinodork

The CI stage should work now if you merge the lastest update to master.

dey4ss avatar Sep 12 '24 06:09 dey4ss

The CI stage should work now if you merge the lastest update to master.

Thanks. All green now!

dinodork avatar Sep 12 '24 08:09 dinodork

@dey4ss Conflicts resolved, please feel free to start the CI workflow.

dinodork avatar Sep 19 '24 14:09 dinodork

Bump.

dinodork avatar Oct 07 '24 09:10 dinodork

Bump #2 @dey4ss

dinodork avatar Oct 22 '24 09:10 dinodork

Hi @dinodork, we currently lack the time to review this PR properly. We would like to merge it, but not without proper reviewing and that might take some more time. Please bear with us.

Bouncner avatar Nov 01 '24 12:11 Bouncner

Hi @dinodork, this PR takes a much higher toll on my brain than anticipated. I am not sure I will be able to properly review it as I am myself not very familiar with parsing.

I just wanted to let you know that we might not be able to merge this PR.

Bouncner avatar Sep 16 '25 11:09 Bouncner