Re: [PR] [SPARK-45686][INFRA][CORE][SQL][SS][CONNECT][MLLIB][DSTREAM][AVRO][ML][K8S][YARN][PYTHON][R][UI][GRAPHX][PROTOBUF][TESTS] Explicitly convert `Array` to `Seq` when function input is defined as `Seq` to avoid compilation warnings related to `class LowPriorityImplicits2 is deprecated` [spark]

["LuciferYang (via GitHub)" <gi...@apache.org>]

LuciferYang commented on PR #43670:
URL: https://github.com/apache/spark/pull/43670#issuecomment-1794757422

   > LGTM if wrap the `Array` with `immutable.ArraySeq` could improve the performance. Otherwise, personally, I accept the compile warnings.
   
   Yes, this can save one collection copy like Scala 2.12, which has potential benefits for memory and performance
   
   


-- 
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-unsubscribe@spark.apache.org

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


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