Skip to content

Commit

Permalink
[doc] Fix CompactionAction doc misspelling (#4238)
Browse files Browse the repository at this point in the history
  • Loading branch information
eric9204 authored Sep 25, 2024
1 parent 9e16ae6 commit baa44fd
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -186,14 +186,14 @@ protected Predicate getPredicate() throws Exception {
predicate = simpleSqlPredicateConvertor.convertSqlToPredicate(whereSql);
}

// Check whether predicate contain non parition key.
// Check whether predicate contain non partition key.
if (predicate != null) {
LOGGER.info("the partition predicate of compaction is {}", predicate);
PartitionPredicateVisitor partitionPredicateVisitor =
new PartitionPredicateVisitor(table.partitionKeys());
Preconditions.checkArgument(
predicate.visit(partitionPredicateVisitor),
"Only parition key can be specialized in compaction action.");
"Only partition key can be specialized in compaction action.");
}

return predicate;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@ public void testUnawareBucketStreamingCompact() throws Exception {

checkLatestSnapshot(table, 2, Snapshot.CommitKind.APPEND);

// repairing that the ut don't specify the real parition of table
// repairing that the ut don't specify the real partition of table
runActionForUnawareTable(true);

// first compaction, snapshot will be 3
Expand Down Expand Up @@ -314,7 +314,7 @@ public void testUnawareBucketBatchCompact() throws Exception {

checkLatestSnapshot(table, 2, Snapshot.CommitKind.APPEND);

// repairing that the ut don't specify the real parition of table
// repairing that the ut don't specify the real partition of table
runActionForUnawareTable(false);

// first compaction, snapshot will be 3.
Expand Down Expand Up @@ -366,7 +366,7 @@ public void testSpecifyNonPartitionField() throws Exception {
rowData(1, 100, 15, BinaryString.fromString("20221209")));

Assertions.assertThatThrownBy(() -> runAction(false))
.hasMessage("Only parition key can be specialized in compaction action.");
.hasMessage("Only partition key can be specialized in compaction action.");
}

@Test
Expand Down

0 comments on commit baa44fd

Please sign in to comment.