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

ZDM-606: Support colon operator in fuzzy SAI queries #112

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Conversation

lukasz-antoniak
Copy link
Contributor

Comment on lines +186 to +191
: K_WHERE relation ( logicalOperator relation )*
;

logicalOperator
: K_AND
| K_OR
Copy link
Collaborator

@joao-r-reis joao-r-reis Jun 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the changes on this file (the highlighted ones here and the lines 219 and 501 below) are basically the only changes to the production code right? just to make sure I'm not missing anything. The rest of the diff is just test code?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And the one below with :. Production code changes were only for syntax definition.

@joao-r-reis
Copy link
Collaborator

Add a CHANGELOG-2.2.md and add this JIRA to that file, then this PR is good to be merged 👍

@lukasz-antoniak lukasz-antoniak force-pushed the ZDM-606 branch 2 times, most recently from ff52e89 to cca35a8 Compare June 6, 2024 14:44
@lukasz-antoniak lukasz-antoniak merged commit c8c257d into main Jun 11, 2024
8 checks passed
@lukasz-antoniak lukasz-antoniak deleted the ZDM-606 branch June 11, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants