ChaladiMohanVamsi opened a new pull request, #8533:
URL: https://github.com/apache/iceberg/pull/8533

   **Context :**
   
   - The `namespaceExists` check in 
[SupportsNamespaces](https://github.com/apache/iceberg/blob/master/api/src/main/java/org/apache/iceberg/catalog/SupportsNamespaces.java#L151-L164)
 behaviour of catalog returns either true or false but doesn't throw 
NoSuchNamespaceException if namespace doesn't exists.
   
   **Usage in Glue Catalog :**
   
   - Below methods of Glue Catalog does `namespaceExists` validation as a 
pre-check.
       - `listTables(Namespace namespace)`
       - `dropNamespace(Namespace namespace)`
   - Current pre-check is no-op as it doesn't take any action when namespace 
doesn't exists.
   
   **PR Contents :**
   
   - This PR attempts to throw `NoSuchNamespaceException` exception when 
`namespaceExists` returns false instead of current no-op behaviour in 
GlueCatalog.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to