sririshindra commented on code in PR #3682:
URL: https://github.com/apache/polaris/pull/3682#discussion_r2785780192


##########
runtime/service/src/main/java/org/apache/polaris/service/config/Serializers.java:
##########
@@ -58,6 +61,7 @@ public static void registerSerializers(ObjectMapper mapper) {
         GrantCatalogRoleRequest.class, new 
GrantCatalogRoleRequestDeserializer());
     module.addDeserializer(AddGrantRequest.class, new 
AddGrantRequestDeserializer());
     module.addDeserializer(RevokeGrantRequest.class, new 
RevokeGrantRequestDeserializer());
+    module.addDeserializer(RegisterTableRequest.class, new 
RegisterTableRequestDeserializer());

Review Comment:
   Removed in https://github.com/apache/polaris/pull/3719



##########
runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/CatalogHandlerUtils.java:
##########
@@ -348,7 +349,21 @@ public LoadTableResponse registerTable(
     request.validate();
 
     TableIdentifier identifier = TableIdentifier.of(namespace, request.name());
-    Table table = catalog.registerTable(identifier, 
request.metadataLocation());
+    // Determine whether the client requested overwrite semantics. For 
catalogs that

Review Comment:
   Removed in https://github.com/apache/polaris/pull/3719



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

Reply via email to