RussellSpitzer commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2587600228
Approved and Merged, thanks @munendrasn for the PR and @Fokko and @nastra
for reviewing
--
This is an automated message from the Apache Git Service.
To respond to the message,
RussellSpitzer merged PR #11260:
URL: https://github.com/apache/iceberg/pull/11260
--
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 unsubscribe, e-mail: issues-unsubscr...@ic
munendrasn commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2587128406
Reached over to iceberg-slack to get the additional review
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
github-actions[bot] commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2585490645
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
nastra commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2538464248
the changes LGTM and given this is only a doc change (by stating a new
property that's being supported) I don't think this PR would have to go through
the normal voting process on the ma
munendrasn commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2538452754
@nastra @Fokko are the changes good to be merged?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
munendrasn commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2497810049
@nastra Could you please review again?
--
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
munendrasn commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2478133424
@nastra Resolved conflicts, please review
--
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
munendrasn commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2476506543
@nastra Could you please take a look.. would this be correct place to add
this info?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
munendrasn commented on PR #11260:
URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2419824121
@nastra Please review, If this looks good, post PR #10722 and
https://github.com/apache/iceberg/pull/10576, will move this change to fileIo
properties (https://github.com/apache/iceb
10 matches
Mail list logo