szehon-ho opened a new pull request, #50511:
URL: https://github.com/apache/spark/pull/50511

   
   
   ### What changes were proposed in this pull request?
   Minor refactoring of new added test to replace unnecessary ArraySeq with 
Array
   
   
   ### Why are the changes needed?
   Rest of the test uses Array, ArraySeq was used in these places as a mistake.
   ArraySeq is also a new scala 2.13 class, I think no reason to use it unless 
it necessary if people want to use Scala 2.12 (although 4.0 branch does not 
need this).
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   Existing unit test
   
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No
   


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to