amogh-jahagirdar merged PR #11779:
URL: https://github.com/apache/iceberg/pull/11779
--
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...@
ebyhr commented on PR #11779:
URL: https://github.com/apache/iceberg/pull/11779#issuecomment-2560579572
@amogh-jahagirdar Thanks for your review. Addressed comments.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
amogh-jahagirdar commented on code in PR #11779:
URL: https://github.com/apache/iceberg/pull/11779#discussion_r1896074073
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -2418,6 +2421,34 @@ public void testPaginationForListTables(int
numberOfItems) {
ebyhr commented on PR #11779:
URL: https://github.com/apache/iceberg/pull/11779#issuecomment-2558801986
@amogh-jahagirdar There is no reason to disagree from my perspective. I was
simply trying to keep the existing behavior because I didn't know the
historical reason. Reverted most changes
