Re: [PR] Core: Relocate parquet to core [iceberg]

2025-01-29 Thread via GitHub
danielcweeks closed pull request #11716: Core: Relocate parquet to core URL: https://github.com/apache/iceberg/pull/11716 -- 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 uns

Re: [PR] Core: Relocate parquet to core [iceberg]

2025-01-29 Thread via GitHub
danielcweeks commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2622982192 Closing in favor of #12060 -- 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: Relocate parquet to core [iceberg]

2025-01-29 Thread via GitHub
ajantha-bhat commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2621425928 Aah, I got the same Spark smoke test failure too. Let me debug this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Core: Relocate parquet to core [iceberg]

2025-01-29 Thread via GitHub
ajantha-bhat commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2621418516 @danielcweeks: I have updated my old PR as per latest code. Please take a look. https://github.com/apache/iceberg/pull/8500 Maybe this can go in for 1.8.0 too. -- Thi

Re: [PR] Core: Relocate parquet to core [iceberg]

2025-01-08 Thread via GitHub
ajantha-bhat commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2578938991 Currently introducing internal writers for parquet: https://github.com/apache/iceberg/pull/11904 Once, it is merged, I will revive/rework the original https://github.com/apa

Re: [PR] Core: Relocate parquet to core [iceberg]

2025-01-08 Thread via GitHub
github-actions[bot] commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2578930131 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] Core: Relocate parquet to core [iceberg]

2024-12-09 Thread via GitHub
ajantha-bhat commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2527881619 @nastra: He kept dummy module for compatibility. But I think it can be removed as mentioned in https://github.com/apache/iceberg/pull/11716#issuecomment-2524744481 My PR ha

Re: [PR] Core: Relocate parquet to core [iceberg]

2024-12-09 Thread via GitHub
nastra commented on PR #11716: URL: https://github.com/apache/iceberg/pull/11716#issuecomment-2527873498 is there anything left in `iceberg-parquet` or can the gradle module be removed too? -- This is an automated message from the Apache Git Service. To respond to the message, please log