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/11/17 19:24:35 UTC
[GitHub] [iceberg] islamismailov opened a new pull request, #6211: Allow dropping a column used by old SortOrders but not current SortOrder
islamismailov opened a new pull request, #6211:
URL: https://github.com/apache/iceberg/pull/6211
Adding a unit test to make sure a column that USED to be a part of a sort order but NO LONGER is, is allowed to be dropped.
--
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] islamismailov commented on pull request #6211: Allow dropping a column used by old SortOrders but not current SortOrder
Posted by GitBox <gi...@apache.org>.
islamismailov commented on PR #6211:
URL: https://github.com/apache/iceberg/pull/6211#issuecomment-1320325932
@Fokko can you have a look? I am just adding a unit test here since it seems like this issue does not affect latest Iceberg.
--
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] Fokko merged pull request #6211: Allow dropping a column used by old SortOrders but not current SortOrder
Posted by GitBox <gi...@apache.org>.
Fokko merged PR #6211:
URL: https://github.com/apache/iceberg/pull/6211
--
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