-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HDDS-2222 Add a method to update ByteBuffer in PureJavaCrc32/PureJavaCrc32C #1578
Conversation
💔 -1 overall
This message was automatically generated. |
+1 for the patch |
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.
+1 for the patch.
The checkstyle warnings in ChecksumByteBuffer are absurd so that we will ignore them. |
The "Switch case falls through" reported by findbugs is by design so that we won't fix it. |
The integration failures definitely are not related since the new code is not yet used in Ozone. |
Oops, I should have added it to findbugsExcludeFile.xml. Otherwise, the warning will keep coming up. Will revert the commit |
Thanks @jnp for reviewing this. Merging now.
This reverts commit 4cf0b36.
Thanks @jnp for reviewing this. Merging now.
This reverts commit 4cf0b36.
See https://issues.apache.org/jira/browse/HDDS-2222