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

add support for postgres drop function statement #1557

Merged
merged 2 commits into from
Jul 6, 2022

Conversation

rrrship
Copy link
Contributor

@rrrship rrrship commented Jun 8, 2022

Adds a support for PostgreSQL DROP FUNCTION statement: PostgreSQL DROP FUNCTION documentation

Also added a new map to Drop class to store parameters by type. Not sure about other drop types, but drop function actually has a more richer syntax with additional parameters which we could support in the future. Storing them in a map could possibly hold off bloating the class.

@manticore-projects
Copy link
Contributor

Greetings.

Thank you very much for your contribution. It looks great, kudos!
@wumpz: Its high quality content and I would like to recommend speedy adoption.

@rrrship
Copy link
Contributor Author

rrrship commented Jun 27, 2022

Since I'm new to open source development and contribution, I wanted to ask when and how often are the pull requests merged to master.

@manticore-projects
Copy link
Contributor

Since I'm new to open source development and contribution, I wanted to ask when and how often are the pull requests merged to master.

Honestly, I do not know either and I am thinking about establishing a "Development Branch", which would merge any pending PRs under a rolling release model.

@wumpz wumpz merged commit 964fa49 into JSQLParser:master Jul 6, 2022
@wumpz
Copy link
Member

wumpz commented Jul 6, 2022

@rrrship sorry about the delay.

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.

3 participants