This is an automated email from the ASF dual-hosted git repository.
danny0405 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 784af0e1786 [HUDI-7514] Update Manifest file after the parquet writer
closed in LSMTimelineWriter (#10883)
784af0e1786 is described below
commit 784af0e17867f249159d4b6040568f3139f91545
Author: Manu <[email protected]>
AuthorDate: Tue Mar 19 18:35:36 2024 +0800
[HUDI-7514] Update Manifest file after the parquet writer closed in
LSMTimelineWriter (#10883)
---
.../java/org/apache/hudi/client/timeline/LSMTimelineWriter.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git
a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/timeline/LSMTimelineWriter.java
b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/timeline/LSMTimelineWriter.java
index 08500f06c9a..e5acf775a19 100644
---
a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/timeline/LSMTimelineWriter.java
+++
b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/timeline/LSMTimelineWriter.java
@@ -124,10 +124,14 @@ public class LSMTimelineWriter {
exceptionHandler.ifPresent(handler -> handler.accept(e));
}
}
- updateManifest(filePath.getName());
} catch (Exception e) {
throw new HoodieCommitException("Failed to write commits", e);
}
+ try {
+ updateManifest(filePath.getName());
+ } catch (Exception e) {
+ throw new HoodieCommitException("Failed to update archiving manifest",
e);
+ }
}
/**
