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/05/27 01:06:52 UTC

[GitHub] [iceberg] kbendick commented on pull request #4880: Core - Use unbound variants of SortOrder / PartitionSpec for REST Catalog's CreateTableRequest

kbendick commented on PR #4880:
URL: https://github.com/apache/iceberg/pull/4880#issuecomment-1139187174

   cc @rdblue @nastra @Fokko.
   
   As mentioned, additional tests that exercise the exact SerDe of these classes are coming, but this patch is necessary to make testing out the REST catalog in its current form easier so I figured we might as well not wait on those other PRs =)
   
   This code is indeed tested via `TestRESTCatalog` and some other places though.


-- 
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