MaxGekk opened a new pull request #24343: [SPARK-26953][CORE][TEST] Disable 
result checking in the test: java.lang.ArrayIndexOutOfBoundsException in TimSort
URL: https://github.com/apache/spark/pull/24343
 
 
   ## What changes were proposed in this pull request?
   
   I propose to disable (comment) result checking in 
`SorterSuite`.`java.lang.ArrayIndexOutOfBoundsException in TimSort` because:
   1. The check is optional, and correctness of TimSort is checked by another 
tests. Purpose of the test is to check that TimSort doesn't fail with 
`ArrayIndexOutOfBoundsException`.
   2. Significantly drops execution time of the test.
   
   Here are timing of running the test locally:
   ```
   Sort: 1.4 seconds
   Result checking: 15.6 seconds
   ``` 
   
   ## How was this patch tested?
   
   By `SorterSuite`.
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to