Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-12-15 Thread via GitHub
github-actions[bot] commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2544225435 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-12-15 Thread via GitHub
github-actions[bot] closed issue #10425: Make ManifestEntry and ManifestReader.liveEntries() as public URL: https://github.com/apache/iceberg/issues/10425 -- 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 g

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-12-01 Thread via GitHub
github-actions[bot] commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2510325692 This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occur

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-06-04 Thread via GitHub
ajantha-bhat commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2147315645 Another PR that requires it to be public: https://github.com/apache/iceberg/pull/10024/files#diff-2cb5b09259db131b4d85059ab8b42a9f3bdfd233bd565539cd78c7f485b44b5dR27 -- This

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-06-03 Thread via GitHub
pudidic commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2146549699 I need to rewrite the files with manifest entry status with details. The details include data file numbers, sequence numbers, and snapshot number, which are not in the DataFile or

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-06-03 Thread via GitHub
amogh-jahagirdar commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2145346311 @pudidic If you want to read the entries wouldn't using the ManifestFiles.read() API be sufficient? Then you could iterate over the data files via the `iterator` API, and

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-06-03 Thread via GitHub
Fokko commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2144916554 I'm reluctant to make it part of the public API. Mostly because you need to have a good understanding of Iceberg to know what you're doing, and probably it is better to go through th

Re: [I] Make ManifestEntry and ManifestReader.liveEntries() as public [iceberg]

2024-06-02 Thread via GitHub
ajantha-bhat commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2144294227 I think initially it was designed that `ManifestFiles` will be local to `iceberg-core` module and we can have a public API's around it that are exposed to other modules.