singhpk234 commented on PR #5860: URL: https://github.com/apache/iceberg/pull/5860#issuecomment-1309542196
> I don't like running 4 tests in a loop inside one method when 2 test methods would do fine. Can you please remove the complexity of parameterizing this test? ACK, removed the loops and added 2 test methods for date partitioned table with datetime java8 enabled and timestamp partitioned table with datetime java8 enabled. -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
