yuqi1129 commented on code in PR #10301:
URL: https://github.com/apache/gravitino/pull/10301#discussion_r2903827656
##########
core/src/main/java/org/apache/gravitino/catalog/TableOperationDispatcher.java:
##########
@@ -402,7 +402,6 @@ public boolean purgeTable(NameIdentifier ident) throws
UnsupportedOperationExcep
store.delete(ident, TABLE);
} catch (NoSuchEntityException e) {
LOG.warn("The table to be purged does not exist in the store: {}",
ident, e);
- return false;
Review Comment:
The problem lies in the method signature `True if the table is purged, false
if the table does not exist.`
`the table` in `false if the table does not exist` refer to table in the
underlying storage, or Gravitino store or both.
--
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]
