mridulm commented on PR #45155: URL: https://github.com/apache/spark/pull/45155#issuecomment-1957639348
Does this PR actually result in an improvement ? `Seq.apply` is expensive only if it is not an indexed seq. The change itself is reasonable, but it looks like current usages pass in an IndexedSeq - and so should not be expensive ? If yes, let us not backport it. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
