rpuch commented on code in PR #4576:
URL: https://github.com/apache/ignite-3/pull/4576#discussion_r1804193258
##########
modules/cluster-management/src/test/java/org/apache/ignite/internal/cluster/management/ClusterStatePersistentSerializerTest.java:
##########
@@ -73,17 +75,19 @@ void serializationAndDeserializationWithNulls() {
@Test
void v1CanBeDeserialized() {
- byte[] bytes =
Base64.getDecoder().decode("Ae++QwMBYQFiAwFjAWQFMy4wLjAHY2x1c3Rlcp1Ct7dR35ELuoeboFbabrgHY29uZmlnAztFBahaoEfJtxGam"
- + "Q6WXJNFRfruL76Bv254dP54iF6V");
+ byte[] bytes =
Base64.getDecoder().decode("Ae++QwMCYgJhAwJkAmMGMy4wLjAIY2x1c3Rlcu/Nq5B4VjQSIUNlhwm63P4HY29uZmlnA+/Nq5B4VhI0IUN"
Review Comment:
Base64 representation was changed because I had to break serialization
format in a backward-incompatible way.
Doing this, I realized that random UUIDs create more problems as I had to
also change asserts on each such change (maybe we'll have to change the format
again before we release 3.0). Fixed UUIDs don't create this problem.
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
