Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DATAJPA-1406 - Improved alias detection. #291

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@schauder
Copy link
Contributor

commented Aug 31, 2018

There is now a minimal parser combinator library available for internal use.
It's used to tokenize SQL statements and then parse the resulting list of tokens.
This identifies the primary alias, join aliases, function aliases, and order by clause.

The new parser should handle comments, character literals, quoted identifiers and subselects correctly.
It also should be easier to extend and to adapt to further requirements.

@schauder schauder force-pushed the master branch from e164784 to 01e36db Nov 1, 2018

schauder added some commits Aug 21, 2018

DATAJPA-1406 - Improved alias detection.
There is now a minimal parser combinator library available for internal use.
It's used to tokenize SQL statements and then parse the resulting list of tokens.
This identifies the primary alias, join aliases, function aliases, and order by claus.

The new parser should handle comments, character literals, quoted identifiers and subselects correctly.
It also should be easier to extend and to adapt to further requirements.

@schauder schauder force-pushed the issue/DATAJPA-1406 branch from 02d742b to 060324c Jul 1, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.