the-other-tim-brown commented on code in PR #13380:
URL: https://github.com/apache/hudi/pull/13380#discussion_r2122427572
##########
hudi-client/hudi-flink-client/src/main/java/org/apache/hudi/table/HoodieFlinkMergeOnReadTable.java:
##########
@@ -117,7 +119,7 @@ public HoodieWriteMetadata<List<WriteStatus>> compact(
HoodieEngineContext context, String compactionInstantTime) {
RunCompactionActionExecutor compactionExecutor = new
RunCompactionActionExecutor(
context, config, this, compactionInstantTime, new
HoodieFlinkMergeOnReadTableCompactor(),
- new HoodieFlinkCopyOnWriteTable(config, context, getMetaClient()),
WriteOperationType.COMPACT);
+ this, WriteOperationType.COMPACT);
Review Comment:
The compaction flow creates the table object right before triggering the
action so it should be fine. The compaction mainly operates off of the plan and
does not need to lookup the fsview or anything like that again.
--
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]
