denis-chudov commented on code in PR #5468:
URL: https://github.com/apache/ignite-3/pull/5468#discussion_r2012042151


##########
modules/catalog/src/main/java/org/apache/ignite/internal/catalog/descriptors/CatalogTableDescriptorSerializers.java:
##########
@@ -191,7 +195,8 @@ static class TableDescriptorSerializerV2 implements 
CatalogObjectSerializer<Cata
         public CatalogTableDescriptor readFrom(CatalogObjectDataInput input) 
throws IOException {
             int id = input.readVarIntAsInt();
             String name = input.readUTF();
-            long updateToken = input.readVarInt();
+            long updateTimestampLong = input.readVarInt();
+            HybridTimestamp updateTimestamp = updateTimestampLong == 0 ? 
MIN_VALUE : hybridTimestamp(updateTimestampLong);

Review Comment:
   It's necessary for correct processing of `0` value that may be found in the 
old storage to convert it to timestamp.



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

Reply via email to