-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
HDFS-16798. SerialNumberMap should decrease current counter if the item exist #4987
Conversation
@ZanderXu Congratulations on your promotion to committer, very good! |
@slfan1989 Thanks, bro. You will get a promotion soon. |
💔 -1 overall
This message was automatically generated. |
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.
Great catch here. +1 from my side.
IMO, this bug will only cause that the capacity of serialNumberMap is less than expected, no other impact, right? If true, suggest to add detail for the description segment of this PR.
@slfan1989 would you mind to take another check? |
LGTM. |
my personal email slfan1989@foxmail.com, looking forward to communicating with you @Hexiaoqiao @ZanderXu |
@Hexiaoqiao @slfan1989 Thanks for your review.
Yes, it is. I have updated the description. |
Committed to trunk. Thanks @ZanderXu for your works. Thanks @slfan1989 for your reviews. |
@ZanderXu Please let me know if we should backport to other active branches. |
…er if the item exist. (apache#4987). Contributed by ZanderXu. Signed-off-by: He Xiaoqiao <hexiaoqiao@apache.org>
This mr may cause the corresponding relationship between t2i and i2t to be inconsistent? @Hexiaoqiao |
Description of PR
During looking into some code related XATTR, I found there is a bug in SerialNumberMap, as bellow:
This bug will only cause that the capacity of serialNumberMap is less than expected, no other impact.