[GitHub] [spark] zhengruifeng commented on pull request #42382: [ML] Remove usage of RDD APIs for load/save in spark-ml

2023-09-20 Thread via GitHub
zhengruifeng commented on PR #42382: URL: https://github.com/apache/spark/pull/42382#issuecomment-1728616265 what about adding a implicit conversions `sc -> spark` in ml: ml only use `spark`; 3-rd lib will need to import this implicit conversion; or just keep two

[GitHub] [spark] zhengruifeng commented on pull request #42382: [ML] Remove usage of RDD APIs for load/save in spark-ml

2023-08-10 Thread via GitHub
zhengruifeng commented on PR #42382: URL: https://github.com/apache/spark/pull/42382#issuecomment-1672974095 yeah, I also feel it may happen to break downstream libraries, even though it is internal change also cc @WeichenXu123 -- This is an automated message from the Apache Git