Skip to content

Commit 197e7fb

Browse files
committed
fix
1 parent 3084017 commit 197e7fb

File tree

4 files changed

+9
-81
lines changed

4 files changed

+9
-81
lines changed

fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/IcebergTransaction.java

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
import org.apache.doris.datasource.ExternalTable;
2525
import org.apache.doris.datasource.NameMapping;
2626
import org.apache.doris.datasource.iceberg.helper.IcebergWriterHelper;
27-
import org.apache.doris.datasource.iceberg.rewrite.RewriteFileInfo;
2827
import org.apache.doris.nereids.trees.plans.commands.insert.IcebergInsertCommandContext;
2928
import org.apache.doris.nereids.trees.plans.commands.insert.InsertCommandContext;
3029
import org.apache.doris.thrift.TIcebergCommitData;
@@ -324,19 +323,6 @@ public long getFilesToAddSize() {
324323
}
325324
}
326325

327-
328-
/**
329-
* Get detailed file information for rewrite operation
330-
*/
331-
public RewriteFileInfo getRewriteFileInfo() {
332-
return new RewriteFileInfo(
333-
getFilesToDeleteCount(),
334-
getFilesToAddCount(),
335-
getFilesToDeleteSize(),
336-
getFilesToAddSize()
337-
);
338-
}
339-
340326
private void commitAppendTxn(List<WriteResult> pendingResults) {
341327
// commit append files.
342328
AppendFiles appendFiles = transaction.newAppend().scanManifestsWith(ops.getThreadPoolWithPreAuth());

fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/rewrite/RewriteFileInfo.java

Lines changed: 0 additions & 62 deletions
This file was deleted.

fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/insert/InsertIntoTableCommand.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -440,8 +440,9 @@ private ExecutorFactory selectInsertExecutorFactory(
440440
dataSink,
441441
physicalSink,
442442
() -> new IcebergInsertExecutor(ctx, icebergExternalTable, label, planner,
443-
Optional.of(icebergInsertCtx),
444-
emptyInsert, jobId)
443+
Optional.of(icebergInsertCtx),
444+
emptyInsert, jobId
445+
)
445446
);
446447
} else if (physicalSink instanceof PhysicalJdbcTableSink) {
447448
boolean emptyInsert = childIsEmptyRelation(physicalSink);

fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/insert/RewriteTableCommand.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,17 +34,17 @@
3434
import org.apache.doris.nereids.properties.PhysicalProperties;
3535
import org.apache.doris.nereids.trees.plans.Explainable;
3636
import org.apache.doris.nereids.trees.plans.Plan;
37+
import org.apache.doris.nereids.trees.plans.PlanType;
38+
import org.apache.doris.nereids.trees.plans.algebra.TVFRelation;
3739
import org.apache.doris.nereids.trees.plans.commands.Command;
3840
import org.apache.doris.nereids.trees.plans.commands.ExplainCommand.ExplainLevel;
3941
import org.apache.doris.nereids.trees.plans.commands.ForwardWithSync;
4042
import org.apache.doris.nereids.trees.plans.commands.NeedAuditEncryption;
41-
import org.apache.doris.nereids.trees.plans.PlanType;
42-
import org.apache.doris.nereids.trees.plans.algebra.TVFRelation;
43-
import org.apache.doris.nereids.trees.plans.visitor.PlanVisitor;
4443
import org.apache.doris.nereids.trees.plans.logical.LogicalPlan;
4544
import org.apache.doris.nereids.trees.plans.physical.PhysicalEmptyRelation;
4645
import org.apache.doris.nereids.trees.plans.physical.PhysicalIcebergTableSink;
4746
import org.apache.doris.nereids.trees.plans.physical.PhysicalSink;
47+
import org.apache.doris.nereids.trees.plans.visitor.PlanVisitor;
4848
import org.apache.doris.nereids.util.RelationUtil;
4949
import org.apache.doris.planner.DataSink;
5050
import org.apache.doris.qe.ConnectContext;
@@ -78,6 +78,9 @@ public class RewriteTableCommand extends Command implements NeedAuditEncryption,
7878
private final Optional<String> branchName;
7979
private long jobId;
8080

81+
/**
82+
* constructor for rewrite operation
83+
*/
8184
public RewriteTableCommand(LogicalPlan logicalQuery, Optional<String> labelName,
8285
Optional<InsertCommandContext> insertCtx, Optional<LogicalPlan> cte, Optional<String> branchName) {
8386
super(PlanType.INSERT_INTO_TABLE_COMMAND);

0 commit comments

Comments
 (0)