You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/07/03 23:28:41 UTC
[GitHub] [iceberg] rdblue opened a new pull request, #5187: Use table partitioning with manual sort order
rdblue opened a new pull request, #5187:
URL: https://github.com/apache/iceberg/pull/5187
This is based on #4922, but adds a check for the unsorted order to make tests pass without modification.
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] rdblue commented on pull request #5187: Core: Use table partitioning with manual sort order
Posted by GitBox <gi...@apache.org>.
rdblue commented on PR #5187:
URL: https://github.com/apache/iceberg/pull/5187#issuecomment-1178358827
Thanks, @aokolnychyi!
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] rdblue commented on pull request #5187: Core: Use table partitioning with manual sort order
Posted by GitBox <gi...@apache.org>.
rdblue commented on PR #5187:
URL: https://github.com/apache/iceberg/pull/5187#issuecomment-1177845066
@aokolnychyi, I updated #4922 to fix the outstanding problems to get it into the release, but it looks like it was updated yesterday. A review would still be helpful to make sure we're happy with the change, on either PR.
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] rdblue merged pull request #5187: Core: Use table partitioning with manual sort order
Posted by GitBox <gi...@apache.org>.
rdblue merged PR #5187:
URL: https://github.com/apache/iceberg/pull/5187
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] aokolnychyi commented on pull request #5187: Core: Use table partitioning with manual sort order
Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on PR #5187:
URL: https://github.com/apache/iceberg/pull/5187#issuecomment-1177126001
I am confused. This PR does not seem to include all changes from #4922. Are both needed then?
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org