singhpk234 commented on issue #12466: URL: https://github.com/apache/iceberg/issues/12466#issuecomment-2714622891
rest spec, just states 404 for table not found : https://github.com/apache/iceberg/blob/main/open-api/rest-catalog-open-api.yaml#L967 But I see where you are coming from you are throwing 400 Bad req with a wrapped exception of NoSuchNamespace ? i think considering we are already at loading the table throwing a namespace not found or not supported exception is gonna be tricky and either case the client need to be aware that such exception can be thrown for making the metadata table work. I would recommend taking this in email-dev list to get opinions on this, and if all agree that this is something thats fair, then add a pr for updating the client. -- 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
