Re: [PR] Core: Make CatalogHandlers.commit public [iceberg]

2024-10-17 Thread via GitHub
nastra closed pull request #9789: Core: Make CatalogHandlers.commit public URL: https://github.com/apache/iceberg/pull/9789 -- 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 u

Re: [PR] Core: Make CatalogHandlers.commit public [iceberg]

2024-02-26 Thread via GitHub
N-o-Z commented on PR #9789: URL: https://github.com/apache/iceberg/pull/9789#issuecomment-1964374624 > @N-o-Z can you elaborate please what exactly you're trying to achieve? Are you trying to implement multi-table commits on the server? For `COMMIT_TRANSACTION` the approach is very simplis

Re: [PR] Core: Make CatalogHandlers.commit public [iceberg]

2024-02-26 Thread via GitHub
nastra commented on PR #9789: URL: https://github.com/apache/iceberg/pull/9789#issuecomment-1964365614 @N-o-Z can you elaborate please what exactly you're trying to achieve? Are you trying to implement multi-table commits on the server? For `COMMIT_TRANSACTION` the approach is very sim

Re: [PR] Core: Make CatalogHandlers.commit public [iceberg]

2024-02-26 Thread via GitHub
N-o-Z commented on PR #9789: URL: https://github.com/apache/iceberg/pull/9789#issuecomment-1964153139 > I believe the purpose was that `CatalogHandlers.commit(..)` should only be used through `CatalogHandlers.updateTable(...)`. Is there a particular issue that prevents you from going throug