Skip to content

/ JSqlParser

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

Jump to bottom

#913 SQL Server variable assignment in a select does not parse | SELECT @var = 1 #914

Closed
wants to merge 2 commits into from
Closed

#913 SQL Server variable assignment in a select does not parse | SELECT @var = 1 #914

wants to merge 2 commits into from

Conversation

@cdandoy
Copy link

@cdandoy cdandoy commented Dec 21, 2019

This fixes #913 but breaks SelectTest.testMultiPartNamesIssue608().
Unfortunately, I don't know JavaCC enough to fix both.

My test works: SELECT @var = 1
but it breaks SELECT @@sessions.tx_read_only
cdandoy added 2 commits Dec 21, 2019
cdandoy
#913 SQL Server variable assignment in a select does not parse | SELECT
Loading status checks…
14ce1be 
@var = 1

This fixes #913 but breaks "SELECT @@sessions.tx_read_only".
Unfortunately, I don't know JavaCC enough to fix both.
cdandoy
#913 SQL Server variable assignment in a select does not parse | SELECT
Loading status checks…
0059489 
@var = 1

Added asserts
@wumpz
Copy link
Member

@wumpz wumpz commented Jul 15, 2020

The function is integrated, but not with your pull request.
@wumpz wumpz closed this Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

SQL Server variable assignment in a select does not parse | SELECT @var = 1
2 participants
@cdandoy @wumpz
You can’t perform that action at this time.