Surya00011 opened a new pull request, #8235: URL: https://github.com/apache/gravitino/pull/8235
# What changes were proposed in this pull request? - Updated `removeCatalog` implementation to respect the `ignoreIfNotExists` flag. - If the catalog does not exist and `ignoreIfNotExists = true`, the method now exits silently. - If the catalog does not exist and `ignoreIfNotExists = false`, a `CatalogException` is thrown. - Wrapped underlying errors in `CatalogException` for consistency. # Why are the changes needed? - The previous implementation ignored the `ignoreIfNotExists` flag. - This led to unexpected exceptions even when users explicitly indicated they wanted missing catalogs to be ignored. - Fixing this ensures more predictable behavior and aligns with the contract of the API. - Fix: #8161 # Does this PR introduce any user-facing change? - **Yes.** - Users who set `ignoreIfNotExists = true` will no longer see unnecessary exceptions if the catalog is missing. - Behavior is now consistent with expected API semantics. -- 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]
