Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-03-10 Thread via GitHub
github-actions[bot] commented on PR #12079: URL: https://github.com/apache/iceberg/pull/12079#issuecomment-2712135448 This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-03-10 Thread via GitHub
github-actions[bot] closed pull request #12079: Parquet: Fix Reader leak by removing useless copy URL: https://github.com/apache/iceberg/pull/12079 -- 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 th

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-03-02 Thread via GitHub
github-actions[bot] commented on PR #12079: URL: https://github.com/apache/iceberg/pull/12079#issuecomment-2692989780 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pul

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-30 Thread via GitHub
zizon commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1936633094 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +75,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-29 Thread via GitHub
zizon commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1935015216 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +75,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-29 Thread via GitHub
zizon commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1935008359 ## parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java: ## @@ -219,6 +228,191 @@ public void testTwoLevelList() throws IOException { assertThat(recor

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-28 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1932439872 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +75,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-28 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1932431503 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +82,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-28 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1932380455 ## parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java: ## @@ -219,6 +228,191 @@ public void testTwoLevelList() throws IOException { assertThat(reco

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-28 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1932375870 ## parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java: ## @@ -219,6 +228,191 @@ public void testTwoLevelList() throws IOException { assertThat(reco

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-28 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1932378730 ## parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java: ## @@ -219,6 +228,191 @@ public void testTwoLevelList() throws IOException { assertThat(reco

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-28 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1932375105 ## parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java: ## @@ -219,6 +228,191 @@ public void testTwoLevelList() throws IOException { assertThat(reco

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-27 Thread via GitHub
zizon commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1930216929 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +82,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-24 Thread via GitHub
zizon commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1929399227 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +82,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-24 Thread via GitHub
nastra commented on code in PR #12079: URL: https://github.com/apache/iceberg/pull/12079#discussion_r1929021070 ## parquet/src/main/java/org/apache/iceberg/parquet/ParquetIO.java: ## @@ -82,22 +82,10 @@ static OutputFile file(org.apache.iceberg.io.OutputFile file, Configuration

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-24 Thread via GitHub
zizon commented on PR #12079: URL: https://github.com/apache/iceberg/pull/12079#issuecomment-2613024665 cc @nastra @Fokko -- 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. T

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-24 Thread via GitHub
zizon commented on PR #12079: URL: https://github.com/apache/iceberg/pull/12079#issuecomment-2612949803 I think I found the root cause. ``` 2025-01-24T10:39:58.963+0800 WARNFinalizer org.apache.iceberg.hadoop.HadoopStreams Unclosed input stream created by: org.ap

Re: [PR] Parquet: Fix Reader leak by removing useless copy [iceberg]

2025-01-24 Thread via GitHub
zizon commented on PR #12079: URL: https://github.com/apache/iceberg/pull/12079#issuecomment-2612281531 Wait, I think the fix is incorrect. The readConf do transfer ownership to FileIterator via init return, in which should be cloused by caller. On Fri, Jan 24, 2025 at 19:10 Edua