ibessonov commented on code in PR #5468:
URL: https://github.com/apache/ignite-3/pull/5468#discussion_r2024773687


##########
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:
   Please state it explicitly in the comment, otherwise it's hard to understand 
what's going on



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