[jira] [Commented] (CASSANDRA-6931) BatchLogManager shouldn't serialize mutations with version 1.2 in 2.1.
[ https://issues.apache.org/jira/browse/CASSANDRA-6931?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13955039#comment-13955039 ] Sylvain Lebresne commented on CASSANDRA-6931: - bq. can't we just special case the 2.1 patch to use the 1.2 version for any node before 2.1 Hum, never mind, that would force us to re-add the serialization code for super columns which is probably more error-prone that not. Well, +1 to the patches then. > BatchLogManager shouldn't serialize mutations with version 1.2 in 2.1. > -- > > Key: CASSANDRA-6931 > URL: https://issues.apache.org/jira/browse/CASSANDRA-6931 > Project: Cassandra > Issue Type: Bug >Reporter: Sylvain Lebresne >Assignee: Aleksey Yeschenko > Fix For: 2.1 beta2 > > > BatchLogManager serialize and deserialize mutations using > MessagingService.VERSION_12 and this is hardcoded. Meaning that it does that > in 2.0, 2.1 and trunk, even though in 2.1 the 1.2 format is not properly > serialized properly since [this > commit|https://github.com/apache/cassandra/commit/cca65d7c1638dcd9370b080f08fd55faefc2733e] > (meaning that I'm pretty sure batch logs on super columns is broken on 2.1 > currently). And keeping the 1.2 format indefinitely just for batchlog is > unrealistic. > So batchlog needs to do something like hints, record the messaging format > used to encode every mutation and use that for deserialization, but always > serialize with the current format. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (CASSANDRA-6931) BatchLogManager shouldn't serialize mutations with version 1.2 in 2.1.
[ https://issues.apache.org/jira/browse/CASSANDRA-6931?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13955038#comment-13955038 ] Sylvain Lebresne commented on CASSANDRA-6931: - It's a bit annoying to have to bump the minimum requirement to upgrade to 2.1 once again: can't we just special case the 2.1 patch to use the 1.2 version for any node before 2.1 (which would make the 2.0 patch unnecessary as a bonus, not that the 2.0 patch is extremely complex)? > BatchLogManager shouldn't serialize mutations with version 1.2 in 2.1. > -- > > Key: CASSANDRA-6931 > URL: https://issues.apache.org/jira/browse/CASSANDRA-6931 > Project: Cassandra > Issue Type: Bug >Reporter: Sylvain Lebresne >Assignee: Aleksey Yeschenko > Fix For: 2.1 beta2 > > > BatchLogManager serialize and deserialize mutations using > MessagingService.VERSION_12 and this is hardcoded. Meaning that it does that > in 2.0, 2.1 and trunk, even though in 2.1 the 1.2 format is not properly > serialized properly since [this > commit|https://github.com/apache/cassandra/commit/cca65d7c1638dcd9370b080f08fd55faefc2733e] > (meaning that I'm pretty sure batch logs on super columns is broken on 2.1 > currently). And keeping the 1.2 format indefinitely just for batchlog is > unrealistic. > So batchlog needs to do something like hints, record the messaging format > used to encode every mutation and use that for deserialization, but always > serialize with the current format. -- This message was sent by Atlassian JIRA (v6.2#6252)