This is an automated email from the ASF dual-hosted git repository.
xushiyan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git
The following commit(s) were added to refs/heads/master by this push:
new 30e89d6438 [MINOR] Fix testUpdateRejectForClustering (#6852)
30e89d6438 is described below
commit 30e89d6438c7a96897040af44cf3177c8d12c248
Author: Zouxxyy <[email protected]>
AuthorDate: Mon Oct 3 13:30:30 2022 +0800
[MINOR] Fix testUpdateRejectForClustering (#6852)
---
.../hudi/client/functional/TestHoodieClientOnCopyOnWriteStorage.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieClientOnCopyOnWriteStorage.java
b/hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieClientOnCopyOnWriteStorage.java
index a0050b3a73..9787387a1e 100644
---
a/hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieClientOnCopyOnWriteStorage.java
+++
b/hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieClientOnCopyOnWriteStorage.java
@@ -1165,7 +1165,7 @@ public class TestHoodieClientOnCopyOnWriteStorage extends
HoodieClientTestBase {
// 3. insert one record with no updating reject exception, and not merge
the small file, just generate a new file group
String commitTime3 = "003";
- insertBatchRecords(client, commitTime3, 1, 1).getKey();
+ insertBatchRecords(client, commitTime3, 1, 1);
List<String> fileGroupIds2 =
table.getFileSystemView().getAllFileGroups(testPartitionPath)
.map(fileGroup ->
fileGroup.getFileGroupId().getFileId()).collect(Collectors.toList());
assertEquals(3, fileGroupIds2.size());
@@ -1178,7 +1178,7 @@ public class TestHoodieClientOnCopyOnWriteStorage extends
HoodieClientTestBase {
String assertMsg = String.format("Not allowed to update the clustering
files in partition: %s "
+ "For pending clustering operations, we are not going to support
update for now.", testPartitionPath);
assertThrows(HoodieUpsertException.class, () -> {
- writeClient.upsert(jsc.parallelize(insertsAndUpdates3, 1),
commitTime3).collect(); }, assertMsg);
+ writeClient.upsert(jsc.parallelize(insertsAndUpdates3, 1),
commitTime4).collect(); }, assertMsg);
// 5. insert one record with no updating reject exception, will merge the
small file
String commitTime5 = "005";