-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
ZOOKEEPER-3971: Auto close resources with try catch block #1507
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this supersedes #1506 and looks okay to me.
By the way, I don't think you necessarily needed to close the old PR and make a new one. You can update the PR by pushing (or force-pushing) to the branch to update the commit log, and you can manually edit the PR ticket title in the web UI.
I've renamed the branch too. |
Ah, yes, if you rename the branch, you have to create a new PR. However, the branch name doesn't matter. That's on the contributor end, not on the receiving end. I'm not sure why the contributing guide recommends using a branch name based on the JIRA number... it doesn't matter... you can name it whatever you want. |
Is there anything left to do for this PR? |
Author: kamaci <furkankamaci@gmail.com> Reviewers: Michael Han <hanm@apache.org>, Enrico Olivelli <eolivelli@gmail.com>, Christopher Tubbs Closes apache#1507 from kamaci/ZOOKEEPER-3971
Author: kamaci <furkankamaci@gmail.com> Reviewers: Michael Han <hanm@apache.org>, Enrico Olivelli <eolivelli@gmail.com>, Christopher Tubbs Closes apache#1507 from kamaci/ZOOKEEPER-3971
Author: kamaci <furkankamaci@gmail.com> Reviewers: Michael Han <hanm@apache.org>, Enrico Olivelli <eolivelli@gmail.com>, Christopher Tubbs Closes apache#1507 from kamaci/ZOOKEEPER-3971
Author: kamaci <furkankamaci@gmail.com> Reviewers: Michael Han <hanm@apache.org>, Enrico Olivelli <eolivelli@gmail.com>, Christopher Tubbs Closes apache#1507 from kamaci/ZOOKEEPER-3971
No description provided.