[GitHub] [spark] MaxGekk edited a comment on pull request #33640: [SPARK-36409][SQL][TESTS] Splitting test cases from datetime.sql

2021-08-05 Thread GitBox
MaxGekk edited a comment on pull request #33640: URL: https://github.com/apache/spark/pull/33640#issuecomment-892888796 -- 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 u

[GitHub] [spark] MaxGekk edited a comment on pull request #33640: [SPARK-36409][SQL][TESTS] Splitting test cases from datetime.sql

2021-08-04 Thread GitBox
MaxGekk edited a comment on pull request #33640: URL: https://github.com/apache/spark/pull/33640#issuecomment-892888796 @cloud-fan Just for my understanding, you moved tests around w/o adding or deleting tests, correct? So, test coverage should be the same. -- This is an automated messag

[GitHub] [spark] MaxGekk edited a comment on pull request #33640: [SPARK-36409][SQL][TESTS] Splitting test cases from datetime.sql

2021-08-04 Thread GitBox
MaxGekk edited a comment on pull request #33640: URL: https://github.com/apache/spark/pull/33640#issuecomment-892888796 @cloud-fan Just for my understanding, you moved tests around w/o adding or deleting tests, correct? And the tests run with the same settings/SQL configs. -- This is an

[GitHub] [spark] MaxGekk edited a comment on pull request #33640: [SPARK-36409][SQL][TESTS] Splitting test cases from datetime.sql

2021-08-04 Thread GitBox
MaxGekk edited a comment on pull request #33640: URL: https://github.com/apache/spark/pull/33640#issuecomment-892888796 @cloud-fan Just for my understanding, you just moved tests around w/o adding or deleting tests, correct? And the tests run with the same settings/SQL configs. -- This