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/09 23:36:01 UTC

[GitHub] [iceberg] singhpk234 commented on pull request #5860: Spark: Fix QueryFailure when running RewriteManifestProcedure on Date partitioned table

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