aweisberg commented on code in PR #3777:
URL: https://github.com/apache/cassandra/pull/3777#discussion_r1917287181


##########
src/java/org/apache/cassandra/tcm/transformations/BeginConsensusMigrationForTableAndRange.java:
##########
@@ -93,24 +85,21 @@ static class Serializer implements 
AsymmetricMetadataSerializer<Transformation,
         public void serialize(Transformation t, DataOutputPlus out, Version 
version) throws IOException
         {
             BeginConsensusMigrationForTableAndRange v = 
(BeginConsensusMigrationForTableAndRange)t;
-            out.writeUTF(v.targetProtocol.toString());
             ConsensusTableMigration.rangesSerializer.serialize(v.ranges, out, 
version);
             serializeCollection(v.tables, out, version, 
TableId.metadataSerializer);
         }
 
         public BeginConsensusMigrationForTableAndRange 
deserialize(DataInputPlus in, Version version) throws IOException
         {
-            ConsensusMigrationTarget targetProtocol = 
ConsensusMigrationTarget.fromString(in.readUTF());

Review Comment:
   We don't have any clusters we need to upgrade yet.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to