zstan commented on code in PR #1296:
URL: https://github.com/apache/ignite-3/pull/1296#discussion_r1012521488
##########
modules/schema/src/main/java/org/apache/ignite/internal/schema/SchemaManager.java:
##########
@@ -301,14 +275,7 @@ private CompletableFuture<SchemaDescriptor>
tableSchema(UUID tblId, String table
CompletableFuture<SchemaDescriptor> fut = new CompletableFuture<>();
if (checkSchemaVersion(tblId, schemaVer)) {
- SchemaDescriptor desc = searchSchemaByVersion(tblId, schemaVer);
-
- if (desc == null) {
- return getSchemaDescriptor(schemaVer, tblCfg);
- } else {
- fut.complete(desc);
- return fut;
- }
+ return getSchemaDescriptor(schemaVer, tblCfg);
Review Comment:
This change is just a revert my previous (possibly erroneous) fix, of course
i can change javadoc (hint me plz a bit what kind of additional info i need to
append) but i pointing that i just revert previous implementation.
--
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]
