[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17250971#comment-17250971 ] Avi Kivity commented on CASSANDRA-13304: I filed

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17250675#comment-17250675 ] Sam Tunnicliffe commented on CASSANDRA-13304: - We can add it for the native protocol

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17250670#comment-17250670 ] Benedict Elliott Smith commented on CASSANDRA-13304: The Hadoop native

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17250630#comment-17250630 ] David Capwell commented on CASSANDRA-13304: --- crc32c is accessible in the class path *

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17250045#comment-17250045 ] Benedict Elliott Smith commented on CASSANDRA-13304: Before you spend ages on

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17249847#comment-17249847 ] Sam Tunnicliffe commented on CASSANDRA-13304: - Thanks [~avi.kivity], that certainly

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17248576#comment-17248576 ] Avi Kivity commented on CASSANDRA-13304: Please consider using the CRC32C polynomial instead

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16735615#comment-16735615 ] Alex Petrov commented on CASSANDRA-13304: - It looks like this patch has introduced protocol

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16606954#comment-16606954 ] Benedict commented on CASSANDRA-13304: -- So, for posterity (and because I have been asked out of

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599420#comment-16599420 ] Jordan West commented on CASSANDRA-13304: - Other thank the bug above I am +1 on the these

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599314#comment-16599314 ] Jordan West commented on CASSANDRA-13304: - [~beobal] I'm still going over the changes but I

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599238#comment-16599238 ] Dinesh Joshi commented on CASSANDRA-13304: -- {{ChecksummingTransformer::transformInbound}} -

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599223#comment-16599223 ] Sam Tunnicliffe commented on CASSANDRA-13304: - thanks both. [~jrwest], I've pulled your

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596966#comment-16596966 ] Jordan West commented on CASSANDRA-13304: - [~beobal] I’ve pushed a patch that converts

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16593106#comment-16593106 ] Jordan West commented on CASSANDRA-13304: - [~beobal] comments below. The protocol comments

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16590918#comment-16590918 ] Dinesh Joshi commented on CASSANDRA-13304: -- Hey [~beobal], Here's my feedback - * I went

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16589081#comment-16589081 ] Sam Tunnicliffe commented on CASSANDRA-13304: - Pushed a couple of updates to : *

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582689#comment-16582689 ] Benedict commented on CASSANDRA-13304: -- It's worth bearing in mind that the TLS variant may be

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582626#comment-16582626 ] Ariel Weisberg commented on CASSANDRA-13304: bq. Is that their expectation? Or that the

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582561#comment-16582561 ] Tom van der Woerdt commented on CASSANDRA-13304: I ran a quick SHA1 benchmark to be

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582544#comment-16582544 ] Benedict commented on CASSANDRA-13304: -- Is that their expectation? Or that the checksum is

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582498#comment-16582498 ] Ariel Weisberg commented on CASSANDRA-13304: If someone just wants a checksum their

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582450#comment-16582450 ] Benedict commented on CASSANDRA-13304: -- Thanks [~tvdw]! I will say that I'm personally +1 this

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16582449#comment-16582449 ] Tom van der Woerdt commented on CASSANDRA-13304: It looks like this can be

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573917#comment-16573917 ] Benedict commented on CASSANDRA-13304: -- Perhaps we should talk through what a general purpose

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573850#comment-16573850 ] Sam Tunnicliffe commented on CASSANDRA-13304: - bq. zstd supports native checksumming too

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573555#comment-16573555 ] Benedict commented on CASSANDRA-13304: -- bq. TLS with a NULL cipher will achieve the same thing

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573499#comment-16573499 ] Aleksey Yeschenko commented on CASSANDRA-13304: --- [~mkjellman] zstd supports native

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572196#comment-16572196 ] Tom van der Woerdt commented on CASSANDRA-13304: My opinion on this hasn't changed:

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572177#comment-16572177 ] Michael Kjellman commented on CASSANDRA-13304: -- we’ve got almost a year of testing on

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572174#comment-16572174 ] Michael Kjellman commented on CASSANDRA-13304: -- and the last patch i did already made

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572170#comment-16572170 ] Michael Kjellman commented on CASSANDRA-13304: -- given we really should be looking to

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572164#comment-16572164 ] Sam Tunnicliffe commented on CASSANDRA-13304: - I've just come back to this with the

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16428628#comment-16428628 ] Jeff Jirsa commented on CASSANDRA-13304: Marking this as a hard blocker for 4.0, because it's

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16183034#comment-16183034 ] Aleksey Yeschenko commented on CASSANDRA-13304: --- Checksums and TLS are orthogonal. It

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16182542#comment-16182542 ] Adam Holmberg commented on CASSANDRA-13304: --- So has it been decided that it's not worth it

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16182003#comment-16182003 ] Michael Kjellman commented on CASSANDRA-13304: -- [~beobal] I've pushed up a trunk rebased

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15967054#comment-15967054 ] Jeff Jirsa commented on CASSANDRA-13304: Vote in favor of making it an opt-out, optional

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15963944#comment-15963944 ] Sylvain Lebresne commented on CASSANDRA-13304: -- bq. defaulting to self-signed

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15963392#comment-15963392 ] Tom van der Woerdt commented on CASSANDRA-13304: +1 for making this optional, as it

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15963238#comment-15963238 ] Michael Kjellman commented on CASSANDRA-13304: -- I've benchmarked this in the real world

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15963109#comment-15963109 ] Adam Holmberg commented on CASSANDRA-13304: --- +1 on making it a startup option, even if it's

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15962679#comment-15962679 ] Alexandre Dutra commented on CASSANDRA-13304: - Added some benchmarks

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15955168#comment-15955168 ] Alexandre Dutra commented on CASSANDRA-13304: - bq. Do you think a ProtocolException is

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15953522#comment-15953522 ] Sam Tunnicliffe commented on CASSANDRA-13304: - Thanks much [~adutra]! bq. Currently

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15949536#comment-15949536 ] Alexandre Dutra commented on CASSANDRA-13304: - I have a compatible Java driver version

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15927715#comment-15927715 ] Sam Tunnicliffe commented on CASSANDRA-13304: - bq.In a brief survey I haven't identified

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15927032#comment-15927032 ] Adam Holmberg commented on CASSANDRA-13304: --- I don't think anyone would argue against

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926980#comment-15926980 ] Joshua McKenzie commented on CASSANDRA-13304: - bq. Regardless, even if the NIC became a

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926927#comment-15926927 ] Michael Kjellman commented on CASSANDRA-13304: -- [~mpenick] I personally have never seen

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926879#comment-15926879 ] Michael Penick commented on CASSANDRA-13304: {quote}Additionally, re: the comment that

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15924898#comment-15924898 ] Michael Kjellman commented on CASSANDRA-13304: -- [~JoshuaMcKenzie] couldn't agree more. I

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15924213#comment-15924213 ] Joshua McKenzie commented on CASSANDRA-13304: - bq. The real downside here would be

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15924210#comment-15924210 ] Sam Tunnicliffe commented on CASSANDRA-13304: - [~mkjellman] thanks for that, responses

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15923239#comment-15923239 ] Michael Kjellman commented on CASSANDRA-13304: -- [~aholmber] There is the hypothetical

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15923191#comment-15923191 ] Michael Kjellman commented on CASSANDRA-13304: -- [~beobal] just finished going thru your

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15907797#comment-15907797 ] Adam Holmberg commented on CASSANDRA-13304: --- I might need a little education. Is there some

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15906751#comment-15906751 ] Michael Kjellman commented on CASSANDRA-13304: -- Yes, it's an actual issue [~tvdw]. TLS

[jira] [Commented] (CASSANDRA-13304) Add checksumming to the native protocol

[ https://issues.apache.org/jira/browse/CASSANDRA-13304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15906708#comment-15906708 ] Tom van der Woerdt commented on CASSANDRA-13304: Do you have any data indicating this