danny0405 commented on code in PR #9617:
URL: https://github.com/apache/hudi/pull/9617#discussion_r1328202576
##########
hudi-cli/src/main/java/org/apache/hudi/cli/commands/ClusteringCommand.java:
##########
@@ -59,7 +58,7 @@ public String scheduleClustering(
SparkLauncher sparkLauncher = SparkUtil.initLauncher(sparkPropertiesPath);
// First get a clustering instant time and pass it to spark launcher for
scheduling clustering
- String clusteringInstantTime = HoodieActiveTimeline.createNewInstantTime();
+ String clusteringInstantTime =
client.getActiveTimeline().createNewInstantTime();
Review Comment:
Kind of think we should avoid to instantiate the active timeline where
possible, the active timeline construction actually triggers the metadata files
listing, which is not relevent with time generation.
--
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]
