IcebreakerSA commented on code in PR #12935:
URL: https://github.com/apache/gravitino/pull/12935#discussion_r4026719875
##########
maintenance/optimizer/src/main/java/org/apache/gravitino/maintenance/optimizer/command/OptimizerCommandContext.java:
##########
@@ -82,6 +83,16 @@ public List<NameIdentifier> parsedIdentifiers() {
return OptimizerCommandUtils.parseIdentifiers(identifiers);
}
+ /**
+ * Parses table identifiers, applying the configured default catalog to
schema-qualified names.
+ *
+ * @return normalized table identifiers
+ */
+ public List<NameIdentifier> parsedTableIdentifiers() {
Review Comment:
Thanks for catching this. Fixed in 4c9b3ee6b: `update-statistics` now uses
normalized table identifiers, and the new regression test verifies that
`schema.table` is passed to the updater as `catalog.schema.table` when a
default catalog is configured. I also clarified the CLI reference to list the
commands that support this shorthand and documented why `append-metrics` does
not apply the default catalog.
--
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]
