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.

Sign up for GitHub

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

[NETBEANS-1603] resolving the problem with (un)showing MySQL procedures within NetBeans IDE DB explorer #5187

Merged
merged 2 commits into from
Jan 10, 2023
Merged

[NETBEANS-1603] resolving the problem with (un)showing MySQL procedures within NetBeans IDE DB explorer #5187

merged 2 commits into from
Jan 10, 2023

Conversation

matthiasblaesing
Copy link
Contributor

Closes: #4945

@matthiasblaesing matthiasblaesing added this to the NB17 milestone Jan 3, 2023
@apache apache locked and limited conversation to collaborators Jan 3, 2023
@apache apache unlocked this conversation Jan 3, 2023
@matthiasblaesing
Copy link
Contributor Author

@sbrbot this PR is a rebase of you change from #4945. I retained your authorship and added a second commit to handle the case when a procedure has no parameters, as indicated in the review comment. I tested this locally with a minimal test DB.

I intend to merge this sometime next week.

@matthiasblaesing matthiasblaesing merged commit 42042ba into apache:master Jan 10, 2023
@matthiasblaesing
Copy link
Contributor Author

@sbrbot thank you for your contribution.

@matthiasblaesing matthiasblaesing deleted the pr-4945 branch January 10, 2023 21:22
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
NB17
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@matthiasblaesing