LuciferYang commented on PR #44041:
URL: https://github.com/apache/spark/pull/44041#issuecomment-1832539867
Thanks @srowen
--
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.
srowen commented on PR #44041:
URL: https://github.com/apache/spark/pull/44041#issuecomment-1832536908
Merged to master
--
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
srowen closed pull request #44041:
[SPARK-46124][CORE][SQL][SS][CONNECT][DSTREAM][MLLIB][ML][PYTHON][R][AVRO][K8S][YARN][UI]
Replace explicit `ArrayOps#toSeq` with
`s.c.immutable.ArraySeq.unsafeWrapArray`
URL: https://github.com/apache/spark/pull/44041
--
This is an automated message from
LuciferYang commented on PR #44041:
URL: https://github.com/apache/spark/pull/44041#issuecomment-1832475933
@srowen this PR Is ready, should we push it forward?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
LuciferYang commented on PR #44041:
URL: https://github.com/apache/spark/pull/44041#issuecomment-183019
Yes, it's ready
--
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.
srowen commented on PR #44041:
URL: https://github.com/apache/spark/pull/44041#issuecomment-1830314897
Seems reasonable to me, continuing the same refactoring. Is it ready?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
LuciferYang commented on PR #44041:
URL: https://github.com/apache/spark/pull/44041#issuecomment-1829307492
cc @srowen, could you help to review this pr if you have time. Thanks
Similar to https://github.com/apache/spark/pull/43666, the purpose of this
pr is also to save a collection
