Re: [PR] [MINOR] Avoid resource leaks [hudi]

2024-01-10 Thread via GitHub
nsivabalan merged PR #10345: URL: https://github.com/apache/hudi/pull/10345 -- 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-unsubscr...@hudi.apa

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2024-01-03 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1874975250 ## CI report: * 76fac0f35c10d1b563229e8807491445f58fa675 Azure: [SUCCESS](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2024-01-02 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1874828489 ## CI report: * c637ed283ad26d4b97d46c7ddedb3858a5744831 Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=2

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2024-01-02 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1874825437 ## CI report: * c637ed283ad26d4b97d46c7ddedb3858a5744831 Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=2

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-28 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870942983 ## CI report: * c637ed283ad26d4b97d46c7ddedb3858a5744831 Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=2

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-27 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870904981 ## CI report: * 03f4555ddfa2cc2861c4624888724d51caca28aa Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-27 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870847172 ## CI report: * b6ebd945a7d980ee5044edcd5b4f8390f6bd8adf Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-27 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870843762 ## CI report: * b6ebd945a7d980ee5044edcd5b4f8390f6bd8adf Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-27 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870798279 ## CI report: * b6ebd945a7d980ee5044edcd5b4f8390f6bd8adf Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-27 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870795572 ## CI report: * b6ebd945a7d980ee5044edcd5b4f8390f6bd8adf Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-27 Thread via GitHub
the-other-tim-brown commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1870778723 > Can this issue be fixed and merged into the 0.14.1 branch as soon as possible? @nsivabalan @danny0405 @the-other-tim-brown It is up to the project committers. I've re-t

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-25 Thread via GitHub
zyclove commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1869288139 Can this issue be fixed and merged into the 0.14.1 branch as soon as possible? @nsivabalan @danny0405 @the-other-tim-brown -- This is an automated message from the Apache Git Service

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
danny0405 commented on code in PR #10345: URL: https://github.com/apache/hudi/pull/10345#discussion_r1429435343 ## hudi-common/src/main/java/org/apache/hudi/common/util/collection/LazyFileIterable.java: ## @@ -128,7 +132,10 @@ private void closeHandle() { } private v

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1859364591 ## CI report: * b6ebd945a7d980ee5044edcd5b4f8390f6bd8adf Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1859284996 ## CI report: * ef6698a802f5ac6752b806f5acc7c15a57ed2f36 Azure: [SUCCESS](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1859283257 ## CI report: * ef6698a802f5ac6752b806f5acc7c15a57ed2f36 Azure: [SUCCESS](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1859249897 ## CI report: * ef6698a802f5ac6752b806f5acc7c15a57ed2f36 Azure: [SUCCESS](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
nsivabalan commented on code in PR #10345: URL: https://github.com/apache/hudi/pull/10345#discussion_r1429207366 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metrics/Metrics.java: ## @@ -112,16 +114,25 @@ private List addAdditionalMetricsExporters(HoodieWrite

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1859213210 ## CI report: * ef6698a802f5ac6752b806f5acc7c15a57ed2f36 Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=21

Re: [PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
hudi-bot commented on PR #10345: URL: https://github.com/apache/hudi/pull/10345#issuecomment-1859211299 ## CI report: * ef6698a802f5ac6752b806f5acc7c15a57ed2f36 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run th

[PR] [MINOR] Avoid resource leaks [hudi]

2023-12-17 Thread via GitHub
the-other-tim-brown opened a new pull request, #10345: URL: https://github.com/apache/hudi/pull/10345 ### Change Logs This PR aims to track down more places there may be leaks in our tests while also cleaning up a couple of spots that were easier to track down. - Added logs to