AMashenkov commented on code in PR #3058: URL: https://github.com/apache/ignite-3/pull/3058#discussion_r1467471178
########## modules/catalog/src/main/java/org/apache/ignite/internal/catalog/descriptors/CatalogTableDescriptor.java: ########## @@ -219,4 +221,53 @@ public void updateToken(long updateToken) { this.creationToken = this.creationToken == INITIAL_CAUSALITY_TOKEN ? updateToken : this.creationToken; } + + /** + * Serializer for {@link CatalogTableDescriptor}. + */ + private static class TableDescriptorSerializer implements CatalogObjectSerializer<CatalogTableDescriptor> { + @Override + public CatalogTableDescriptor readFrom(int version, IgniteDataInput input) throws IOException { + CatalogDescriptorBase header = CatalogObjectDescriptor.SERIALIZER.readFrom(version, input); Review Comment: Let's get rid of additional objects. ```suggestion int id = input.readInt() String name = input.readUtf(); long input = readLong(); ``` -- 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