danielcweeks commented on issue #123:
URL: https://github.com/apache/iceberg-python/issues/123#issuecomment-1791411464

   Other than the obvious edge case: loading a table from a catalog that 
supports 4-part namespaces and happens to have a namespace that matchings the 
first part.
   
   I would propose that we just update `load_catalog` to check if the first 
part matches the current catalog (and has at least three parts) and remove it.  
The java implementation doesn't currently provide access to the identifier, so 
there's currently no equivalent problem.  I believe the table also has 
reference to the catalog, so we could also check that the catalog is the 
correct instance of the catalog.
   
   Thoughts @Fokko ?
   
   


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to