-
Notifications
You must be signed in to change notification settings - Fork 13k
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
[FLINK-1766]Fix the bug of equals function of FSKey #511
Conversation
Looks good. This seems more like an optimization, than a bug, Before we can merge the change, it needs to be brought in sync with the code style. Currently, the checkstyle rules reject the code for using spaces rather than tabs. |
If you are interested: Here are some simple guidelines on how to format code and create easy to merge commits for pull requests: |
Great thanks to Ewen. I will first have a study of how to format code. Then pull a request. |
Thanks for Ewen. That help me a lot to understand how to contribute to Flink. I will follow the guide in the future. |
No problem. We are happy that you are interested to contribute :-) |
@StephanEwen I have fixed the code style issue and changed the PR to an "Improvement" one in jira. |
LGTM. Will merge later today. Thanks! |
The equals function in org.apache.flink.core.fs.FileSystem.FSKey should first confirm whether obj == this, if obj is the same object.It should return true Author: hongsibao <hongsibao@huawei.com> Closes apache#511 from matadorhong/Bug_Flink_1776 and squashes the following commits: 1ad06d7 [hongsibao] Fix the code format problem 431fc4b [hongsibao] Fix the code format problem ab0ae5e [hongsibao] [FLINK-1766]Fix the bug of equals function of FSKey
The equals function in org.apache.flink.core.fs.FileSystem.FSKey should first confirm whether obj == this, if obj is the same object.It should return true Author: hongsibao <hongsibao@huawei.com> Closes apache#511 from matadorhong/Bug_Flink_1776 and squashes the following commits: 1ad06d7 [hongsibao] Fix the code format problem 431fc4b [hongsibao] Fix the code format problem ab0ae5e [hongsibao] [FLINK-1766]Fix the bug of equals function of FSKey
The equals function in org.apache.flink.core.fs.FileSystem.FSKey should first confirm whether obj == this, if obj is the same object.It should return true