Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2025-03-04 Thread via GitHub
ajantha-bhat commented on PR #8500: URL: https://github.com/apache/iceberg/pull/8500#issuecomment-2697044257 Closing this as community is favoring[1] towards using parquet via reflection in core module instead of moving files from parquet to core. [1] https://github.com/apache/iceber

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2025-03-04 Thread via GitHub
ajantha-bhat closed pull request #8500: Core: Move `iceberg-parquet` files to `iceberg-core` URL: https://github.com/apache/iceberg/pull/8500 -- 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 spec

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2025-02-28 Thread via GitHub
github-actions[bot] commented on PR #8500: URL: https://github.com/apache/iceberg/pull/8500#issuecomment-2691756916 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 pull

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2025-01-29 Thread via GitHub
ajantha-bhat commented on code in PR #8500: URL: https://github.com/apache/iceberg/pull/8500#discussion_r1933732242 ## build.gradle: ## @@ -897,7 +874,9 @@ project(':iceberg-snowflake') { } dependencies { -implementation project(':iceberg-core') +implementation(p

[PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2025-01-29 Thread via GitHub
ajantha-bhat opened a new pull request, #8500: URL: https://github.com/apache/iceberg/pull/8500 - Since core module need to write stats in parquet format, to avoid circular dependency, move all the files from iceberg-parquet module to iceberg code. Fixes #8453 -- This is an automa

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2023-11-27 Thread via GitHub
ajantha-bhat closed pull request #8500: Core: Move `iceberg-parquet` files to `iceberg-core` URL: https://github.com/apache/iceberg/pull/8500 -- 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 spec

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2023-11-27 Thread via GitHub
ajantha-bhat commented on PR #8500: URL: https://github.com/apache/iceberg/pull/8500#issuecomment-1829224667 Going with alternative approach of writing stats from `iceberg-data` module and engines calling that. Core API will not write stats. -- This is an automated message from the Ap

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2023-11-02 Thread via GitHub
ajantha-bhat commented on code in PR #8500: URL: https://github.com/apache/iceberg/pull/8500#discussion_r1380176383 ## api/src/test/java/org/apache/iceberg/TestHelpers.java: ## @@ -265,6 +266,18 @@ public static void serialize(final Serializable obj, final OutputStream outputSt

Re: [PR] Core: Move `iceberg-parquet` files to `iceberg-core` [iceberg]

2023-11-02 Thread via GitHub
ajantha-bhat commented on code in PR #8500: URL: https://github.com/apache/iceberg/pull/8500#discussion_r1380176383 ## api/src/test/java/org/apache/iceberg/TestHelpers.java: ## @@ -265,6 +266,18 @@ public static void serialize(final Serializable obj, final OutputStream outputSt