You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2017/12/23 01:47:10 UTC

[GitHub] sijie opened a new pull request #917: Issue 915: Part 2 - Add crc32c integer/long checksum utils under `com.scurrilous.circe.checksum`

sijie opened a new pull request #917: Issue 915: Part 2 - Add crc32c integer/long checksum utils under `com.scurrilous.circe.checksum`
URL: https://github.com/apache/bookkeeper/pull/917
 
 
   Descriptions of the changes in this PR:
   
   This is the part 2 change of #915. It is based on #916, adding crc32c int/long checksum util classes under `com.scurrilous.circe.checksum`. It also fixes a incremental checksum issue on long crc32c. I added bunch of test cases to ensure the both long/int crc32 checksum is correct on incremental cases.
   
   a7a26e1  contains the changes to be reviewed in this pull request.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services