Skip to content

[CALCITE-2353] Allow user to override SqlSetOption#722

Closed
apilloud wants to merge 1 commit intoapache:masterfrom
apilloud:sqlset
Closed

[CALCITE-2353] Allow user to override SqlSetOption#722
apilloud wants to merge 1 commit intoapache:masterfrom
apilloud:sqlset

Conversation

@apilloud
Copy link
Member

@apilloud apilloud commented Jun 5, 2018

If alterStatementParserMethods are added before SqlSetOption then the user can replace SqlSetOption. This change matches how statementParserMethods are added at the top level.

Copy link

@suez1224 suez1224 left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the PR. It looks good to me. Will merge soon.

@asfgit asfgit closed this in aaf4471 Jun 8, 2018
wangxlong pushed a commit to wangxlong/calcite that referenced this pull request Feb 13, 2020
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Jun 18, 2021
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Jun 18, 2021
Close apache#722

Change-Id: I3da0f1d5b5cafc9ab0d53832afdeebe1789e2461
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Aug 28, 2025
Close apache#722

Change-Id: I3da0f1d5b5cafc9ab0d53832afdeebe1789e2461
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