Skip to content

Commit 2a9efdf

Browse files
committed
[core] Fix in CompactProcedure
1 parent e88c6e9 commit 2a9efdf

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

paimon-core/src/main/java/org/apache/paimon/append/AppendCompactTask.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@
3535

3636
import java.util.ArrayList;
3737
import java.util.Collections;
38-
import java.util.Comparator;
3938
import java.util.List;
4039
import java.util.Objects;
4140

@@ -50,7 +49,6 @@ public class AppendCompactTask {
5049

5150
public AppendCompactTask(BinaryRow partition, List<DataFileMeta> files) {
5251
Preconditions.checkArgument(files != null);
53-
files.sort(Comparator.comparingLong(DataFileMeta::minSequenceNumber));
5452
this.partition = partition;
5553
compactBefore = new ArrayList<>(files);
5654
compactAfter = new ArrayList<>();

paimon-spark/paimon-spark-common/src/main/java/org/apache/paimon/spark/procedure/CompactProcedure.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@
8787
import java.util.ArrayList;
8888
import java.util.Arrays;
8989
import java.util.Collections;
90+
import java.util.Comparator;
9091
import java.util.HashMap;
9192
import java.util.Iterator;
9293
import java.util.List;
@@ -459,6 +460,13 @@ private void compactUnAwareBucketTable(
459460
ser.deserialize(
460461
ser.getVersion(),
461462
taskIterator.next());
463+
if (coreOptions.rowTrackingEnabled()) {
464+
task.compactBefore()
465+
.sort(
466+
Comparator.comparingLong(
467+
DataFileMeta
468+
::minSequenceNumber));
469+
}
462470
messages.add(
463471
messageSer.serialize(
464472
task.doCompact(table, write)));

0 commit comments

Comments
 (0)