This is an automated email from the ASF dual-hosted git repository.
jmclean pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push:
new be3f7a39a1 [#8346] fix(server): fix misleading log message in catalog
drop operation (#8351)
be3f7a39a1 is described below
commit be3f7a39a1d12f6b4f5c43844c7fafd8300a779f
Author: Kyle Lin <[email protected]>
AuthorDate: Fri Aug 29 15:23:04 2025 +0800
[#8346] fix(server): fix misleading log message in catalog drop operation
(#8351)
### What changes were proposed in this pull request?
Move success log message inside conditional block to avoid misleading
logs in `CatalogOperations.java` .
### Why are the changes needed?
Fixes #8346
### Does this PR introduce any user-facing change?
No.
### How was this patch tested?
- Ran `./gradlew clean build`
---
.../java/org/apache/gravitino/server/web/rest/CatalogOperations.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
a/server/src/main/java/org/apache/gravitino/server/web/rest/CatalogOperations.java
b/server/src/main/java/org/apache/gravitino/server/web/rest/CatalogOperations.java
index f7461bdf56..68a8a56f2e 100644
---
a/server/src/main/java/org/apache/gravitino/server/web/rest/CatalogOperations.java
+++
b/server/src/main/java/org/apache/gravitino/server/web/rest/CatalogOperations.java
@@ -333,9 +333,10 @@ public class CatalogOperations {
boolean dropped = catalogDispatcher.dropCatalog(ident, force);
if (!dropped) {
LOG.warn("Failed to drop catalog {} under metalake {}",
catalogName, metalakeName);
+ } else {
+ LOG.info("Catalog dropped: {}.{}", metalakeName, catalogName);
}
Response response = Utils.ok(new DropResponse(dropped));
- LOG.info("Catalog dropped: {}.{}", metalakeName, catalogName);
return response;
});
} catch (Exception e) {
