You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "cshuo (via GitHub)" <gi...@apache.org> on 2026/09/28 03:09:04 UTC
[PR] fix(common): close native log writers independently on failure [hudi]
cshuo opened a new pull request, #20118:
URL: https://github.com/apache/hudi/pull/20118
### Describe the issue this Pull Request addresses
Closes #20117.
When native MOR logs contain both data and delete writers, a failure while closing the data writer or reading its file-format metadata skips delete writer closure and state cleanup. A delete writer close failure also leaves stale state behind.
### Summary and Changelog
- Ensure `HoodieNativeLogFormatWriter` attempts both writer closes, using `CloseableUtils.closeSuppressing()` to preserve the first failure and attach subsequent close failures.
- Clear each writer reference and its record positions in `finally`, after attempting that writer's close.
- Add 16 regression cases covering `close()` and `flushAppend()`, close and metadata failures, suppressed exceptions, repeated close calls, and record-position cleanup.
Validation: `TestHoodieNativeLogFormatWriter` passes all 28 tests, with no failures, errors, or skips. Checkstyle passes.
```bash
mvn -pl hudi-client/hudi-client-common -am \
-Dtest=TestHoodieNativeLogFormatWriter \
-Dsurefire.failIfNoSpecifiedTests=false \
-DskipITs -DskipSparkTests -DskipScalaTests test
```
### Impact
Native log writers release resources and clear pending state even when close or metadata retrieval fails. The original exception is retained, and repeated close calls do not retry already-attempted writers. No public API or configuration changes.
### Risk Level
Low. The change is limited to native log writer cleanup and exception handling. Targeted tests cover both entry points and verify that subsequent appends do not inherit stale record positions.
### Documentation Update
None.
### Contributor's checklist
- [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute)
- [x] Enough context is provided in the sections above
- [x] Adequate tests were added if applicable
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5883093019
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "149f16da800ce05f134aa7442e57871852eb93e8",
"status" : "SUCCESS",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323",
"triggerID" : "149f16da800ce05f134aa7442e57871852eb93e8",
"triggerType" : "PUSH"
}, {
"hash" : "d69409787905d2454454e34a5220730d35fb0c12",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "d69409787905d2454454e34a5220730d35fb0c12",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 149f16da800ce05f134aa7442e57871852eb93e8 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323)
* d69409787905d2454454e34a5220730d35fb0c12 UNKNOWN
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5883161830
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "149f16da800ce05f134aa7442e57871852eb93e8",
"status" : "SUCCESS",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323",
"triggerID" : "149f16da800ce05f134aa7442e57871852eb93e8",
"triggerType" : "PUSH"
}, {
"hash" : "d69409787905d2454454e34a5220730d35fb0c12",
"status" : "PENDING",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17341",
"triggerID" : "d69409787905d2454454e34a5220730d35fb0c12",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 149f16da800ce05f134aa7442e57871852eb93e8 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323)
* d69409787905d2454454e34a5220730d35fb0c12 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17341)
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5862715728
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/20118?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:x: Patch coverage is `70.58824%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 80.40%. Comparing base ([`ead194c`](https://app.codecov.io/gh/apache/hudi/commit/ead194cce54829fae9113689a2f28772d0d76045?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`149f16d`](https://app.codecov.io/gh/apache/hudi/commit/149f16da800ce05f134aa7442e57871852eb93e8?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/20118?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...pache/hudi/io/cdc/HoodieNativeLogFormatWriter.java](https://app.codecov.io/gh/apache/hudi/pull/20118?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fio%2Fcdc%2FHoodieNativeLogFormatWriter.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2lvL2NkYy9Ib29kaWVOYXRpdmVMb2dGb3JtYXRXcml0ZXIuamF2YQ==) | 70.58% | [5 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/20118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #20118 +/- ##
============================================
- Coverage 80.40% 80.40% -0.01%
- Complexity 34874 34875 +1
============================================
Files 2546 2546
Lines 142889 142896 +7
Branches 17374 17373 -1
============================================
+ Hits 114895 114897 +2
- Misses 20082 20087 +5
Partials 7912 7912
```
| [Components](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=components&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [hudi-common](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.97% <ø> (ø)` | |
| [hudi-client](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.43% <70.58%> (-0.02%)` | :arrow_down: |
| [hudi-flink](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `85.74% <ø> (ø)` | |
| [hudi-spark-datasource](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `73.85% <ø> (ø)` | |
| [hudi-utilities](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `78.17% <ø> (ø)` | |
| [hudi-cli](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.05% <ø> (ø)` | |
| [hudi-hadoop](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.99% <ø> (ø)` | |
| [hudi-sync](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `76.02% <ø> (ø)` | |
| [hudi-io](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `81.50% <ø> (ø)` | |
| [hudi-timeline-service](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.06% <ø> (ø)` | |
| [hudi-cloud](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `81.00% <ø> (ø)` | |
| [hudi-kafka-connect](https://app.codecov.io/gh/apache/hudi/pull/20118/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `53.20% <ø> (ø)` | |
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/20118/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/20118/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `43.99% <70.58%> (+<0.01%)` | :arrow_up: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/20118?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...pache/hudi/io/cdc/HoodieNativeLogFormatWriter.java](https://app.codecov.io/gh/apache/hudi/pull/20118?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fio%2Fcdc%2FHoodieNativeLogFormatWriter.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2lvL2NkYy9Ib29kaWVOYXRpdmVMb2dGb3JtYXRXcml0ZXIuamF2YQ==) | `86.80% <70.58%> (-2.98%)` | :arrow_down: |
</details>
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5863947789
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "149f16da800ce05f134aa7442e57871852eb93e8",
"status" : "SUCCESS",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323",
"triggerID" : "149f16da800ce05f134aa7442e57871852eb93e8",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 149f16da800ce05f134aa7442e57871852eb93e8 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323)
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "danny0405 (via GitHub)" <gi...@apache.org>.
danny0405 merged PR #20118:
URL: https://github.com/apache/hudi/pull/20118
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-agent (via GitHub)" <gi...@apache.org>.
hudi-agent commented on code in PR #20118:
URL: https://github.com/apache/hudi/pull/20118#discussion_r4129324415
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/cdc/HoodieNativeLogFormatWriter.java:
##########
@@ -277,27 +278,41 @@ private void ensureAppendVersion() throws IOException {
}
private void closeFileWriters() throws IOException {
- if (dataFileWriter != null) {
- dataFileWriter.close();
- if (writeConfig.isMetadataColumnStatsIndexEnabled()) {
- try {
- lastDataFileFormatMetadata = Option.ofNullable(dataFileWriter.getFileFormatMetadata());
- } catch (UnsupportedOperationException e) {
- // File-format metadata is an optional writer capability. Unsupported formats can still produce valid
- // native log files; they simply do not contribute column stats for this append.
+ try {
+ try {
+ if (dataFileWriter != null) {
lastDataFileFormatMetadata = Option.empty();
+ dataFileWriter.close();
+ if (writeConfig.isMetadataColumnStatsIndexEnabled()) {
+ try {
+ lastDataFileFormatMetadata = Option.ofNullable(dataFileWriter.getFileFormatMetadata());
+ } catch (UnsupportedOperationException e) {
+ // File-format metadata is an optional writer capability. Unsupported formats can still produce valid
+ // native log files; they simply do not contribute column stats for this append.
+ lastDataFileFormatMetadata = Option.empty();
+ }
+ }
}
- } else {
- lastDataFileFormatMetadata = Option.empty();
+ } finally {
+ dataFileWriter = null;
Review Comment:
🤖 This can happen in production, not just in the test. When a rollover `flushAppend()` fails inside `writeInsertAndUpdate`/`writeDelete`, `HoodieAppendHandle#writeRecord` catches it. If `hoodie.write.ignore.failed` is on (the Spark default is `true`), it marks that one record as failed and keeps going instead of closing the writer. Now that the writers are nulled, `canWriteDataFile()` returns true on the next record. `ensureAppendVersion()` does nothing because `currentAppendVersion` is still set, so `createNativeLogFile` builds the same version and write-token path as the file whose close failed. Resetting `currentAppendVersion`, `dataLogFile` and `deleteLogFile` in a `finally` in `flushAppend` (or making the writer fail fast after a failed flush) would avoid a collision with or overwrite of that partial file.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-agent (via GitHub)" <gi...@apache.org>.
hudi-agent commented on code in PR #20118:
URL: https://github.com/apache/hudi/pull/20118#discussion_r4127205232
##########
hudi-client/hudi-client-common/src/test/java/org/apache/hudi/io/cdc/TestHoodieNativeLogFormatWriter.java:
##########
@@ -208,6 +216,106 @@ public void testSkipsDataFileFormatMetadataWhenColumnStatsDisabled() throws Exce
assertFalse(metadata.isPresent());
}
+ @ParameterizedTest
+ @CsvSource({
+ "io, false, false", "io, false, true",
+ "io, true, false", "io, true, true",
+ "runtime, true, false", "runtime, true, true",
Review Comment:
🤖 nit: the failureMode strings ("io", "runtime", "metadata", "error", "none", "shared") are magic values whose meaning only becomes clear once you read the method body. Consider a small enum or at least a comment near the @CsvSource block explaining what each mode simulates.
<sub><i>⚠️ AI-generated; verify before applying. React 👍/👎 to flag quality.</i></sub>
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/cdc/HoodieNativeLogFormatWriter.java:
##########
@@ -277,27 +278,41 @@ private void ensureAppendVersion() throws IOException {
}
private void closeFileWriters() throws IOException {
- if (dataFileWriter != null) {
- dataFileWriter.close();
- if (writeConfig.isMetadataColumnStatsIndexEnabled()) {
- try {
- lastDataFileFormatMetadata = Option.ofNullable(dataFileWriter.getFileFormatMetadata());
- } catch (UnsupportedOperationException e) {
- // File-format metadata is an optional writer capability. Unsupported formats can still produce valid
- // native log files; they simply do not contribute column stats for this append.
+ try {
+ try {
+ if (dataFileWriter != null) {
lastDataFileFormatMetadata = Option.empty();
+ dataFileWriter.close();
+ if (writeConfig.isMetadataColumnStatsIndexEnabled()) {
+ try {
+ lastDataFileFormatMetadata = Option.ofNullable(dataFileWriter.getFileFormatMetadata());
+ } catch (UnsupportedOperationException e) {
+ // File-format metadata is an optional writer capability. Unsupported formats can still produce valid
+ // native log files; they simply do not contribute column stats for this append.
+ lastDataFileFormatMetadata = Option.empty();
+ }
+ }
}
- } else {
- lastDataFileFormatMetadata = Option.empty();
+ } finally {
+ dataFileWriter = null;
Review Comment:
🤖 Now that the writers are nulled even when `flushAppend()` fails, the next append will open a new writer. But `flushAppend` throws before it resets `currentAppendVersion`, `dataLogFile` and `deleteLogFile`, so that new writer would reuse the same version and path as the file whose close just failed. Is reusing the writer after a failed flush meant to be supported (the new test does this)? If so, should the failure path also reset `currentAppendVersion` so we don't overwrite or collide with a partially written log file?
<sub><i>⚠️ AI-generated; verify before applying. React 👍/👎 to flag quality.</i></sub>
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/cdc/HoodieNativeLogFormatWriter.java:
##########
@@ -277,27 +278,41 @@ private void ensureAppendVersion() throws IOException {
}
private void closeFileWriters() throws IOException {
- if (dataFileWriter != null) {
- dataFileWriter.close();
- if (writeConfig.isMetadataColumnStatsIndexEnabled()) {
- try {
- lastDataFileFormatMetadata = Option.ofNullable(dataFileWriter.getFileFormatMetadata());
- } catch (UnsupportedOperationException e) {
- // File-format metadata is an optional writer capability. Unsupported formats can still produce valid
- // native log files; they simply do not contribute column stats for this append.
+ try {
Review Comment:
🤖 nit: closeFileWriters now nests try/try-finally/catch three levels deep, which makes the control flow (especially where lastDataFileFormatMetadata gets reset) harder to follow. Might be worth extracting the data-writer-close logic into its own `closeDataFileWriter()` method (mirroring the new `closeDeleteFileWriter()`) so this method just reads as "close data, suppress-close delete on failure".
<sub><i>⚠️ AI-generated; verify before applying. React 👍/👎 to flag quality.</i></sub>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5862814704
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "149f16da800ce05f134aa7442e57871852eb93e8",
"status" : "PENDING",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323",
"triggerID" : "149f16da800ce05f134aa7442e57871852eb93e8",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 149f16da800ce05f134aa7442e57871852eb93e8 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323)
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5884274718
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "149f16da800ce05f134aa7442e57871852eb93e8",
"status" : "DELETED",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17323",
"triggerID" : "149f16da800ce05f134aa7442e57871852eb93e8",
"triggerType" : "PUSH"
}, {
"hash" : "d69409787905d2454454e34a5220730d35fb0c12",
"status" : "SUCCESS",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17341",
"triggerID" : "d69409787905d2454454e34a5220730d35fb0c12",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* d69409787905d2454454e34a5220730d35fb0c12 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=17341)
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(common): close native log writers independently on failure [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #20118:
URL: https://github.com/apache/hudi/pull/20118#issuecomment-5862737092
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "149f16da800ce05f134aa7442e57871852eb93e8",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "149f16da800ce05f134aa7442e57871852eb93e8",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 149f16da800ce05f134aa7442e57871852eb93e8 UNKNOWN
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org