FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: Add support for `REFRESH MATERIALIZED VIEW` by jxnu-liguobin · Pull Request #1911 · JSQLParser/JSqlParser · GitHub

feat: Add support for REFRESH MATERIALIZED VIEW - #1911

Merged
manticore-projects merged 12 commits into
JSQLParser:masterfrom
jxnu-liguobin:support-materialized-view
Dec 11, 2023
Merged

feat: Add support for REFRESH MATERIALIZED VIEW#1911
manticore-projects merged 12 commits into
JSQLParser:masterfrom
jxnu-liguobin:support-materialized-view

Conversation

Copy link
Copy Markdown
Contributor

closed #1692

jxnu-liguobin marked this pull request as ready for review December 8, 2023 03:01

Copy link
Copy Markdown
Contributor

Greetings! Thank your for your contribution and welcome to the JSQLParser team.

manticore-projects commented Dec 9, 2023
edited
Loading

Copy link
Copy Markdown
Contributor

You added new Tokens and so you will need to update the Keywords as described here: https://manticore-projects.com/JSQLParser/contribution.html#manage-reserved-keywords

Also you will need to synchronize against latest commit since I amended the Grammar. Sorry for this hassle, there was no better way to incorporate needed corrections.

jxnu-liguobin force-pushed the support-materialized-view branch from 77c66eb to c5ba0f1 Compare December 11, 2023 01:13
jxnu-liguobin changed the title Support REFRESH MATERIALIZED VIEW feat: Add support for REFRESH MATERIALIZED VIEW Dec 11, 2023

Copy link
Copy Markdown
Contributor Author

You added new Tokens and so you will need to update the Keywords as described here: https://manticore-projects.com/JSQLParser/contribution.html#manage-reserved-keywords

Also you will need to synchronize against latest commit since I amended the Grammar. Sorry for this hassle, there was no better way to incorporate needed corrections.

I modified the PR, how about it?

manticore-projects merged commit 425c72e into JSQLParser:master Dec 11, 2023

Copy link
Copy Markdown
Contributor

I modified the PR, how about it?

Kudos, very nicely done! Thank you for your contribution and look forward seeing more PRs from you soon.

Copy link
Copy Markdown
Contributor Author

I modified the PR, how about it?

Kudos, very nicely done! Thank you for your contribution and look forward seeing more PRs from you soon.

Thank you, when is the next version expected to be released?

Copy link
Copy Markdown
Contributor

Thank you, when is the next version expected to be released?
@wumpz: I think we have the perfect material for a smooth End of Year Release, please consider a mvn publish as soon as you can.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

"REFRESH MATERIALIZED VIEW (...)" is not supported

2 participants


Back | FazBrowse Home | New Git URL