[jira] [Commented] (KAFKA-3174) Re-evaluate the CRC32 class performance.

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/KAFKA-3174?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16300819#comment-16300819 ] 

ASF GitHub Bot commented on KAFKA-3174:
---------------------------------------

ijuma commented on issue #841: KAFKA-3174: Change Crc32 to use java.util.zip.CRC32
URL: https://github.com/apache/kafka/pull/841#issuecomment-353500266
 
 
   This is less important since message format 2 is using CRC32C. I'm going to close this, but feel free to reopen if you think we should still consider this change.

----------------------------------------------------------------
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


> Re-evaluate the CRC32 class performance.
> ----------------------------------------
>
>                 Key: KAFKA-3174
>                 URL: https://issues.apache.org/jira/browse/KAFKA-3174
>             Project: Kafka
>          Issue Type: Improvement
>    Affects Versions: 0.9.0.0
>            Reporter: Jiangjie Qin
>            Assignee: Jiangjie Qin
>
> We used org.apache.kafka.common.utils.CRC32 in clients because it has better performance than java.util.zip.CRC32 in Java 1.6.
> In a recent test I ran it looks in Java 1.8 the CRC32 class is 2x as fast as the Crc32 class we are using now. We may want to re-evaluate the performance of Crc32 class and see it makes sense to simply use java CRC32 instead.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)