holdenk commented on pull request #29788:
URL: https://github.com/apache/spark/pull/29788#issuecomment-698021508
I think it would be good to see your proposal in code @Ngone51 because I'm
not 100% sure what you mean.
I would really like to see both this and the precursor tested more
holdenk commented on pull request #29788:
URL: https://github.com/apache/spark/pull/29788#issuecomment-695556053
(willing to switch back to -0 once the original issue is addressed, I just
don't want us in a state with broken tests as normal).
holdenk commented on pull request #29788:
URL: https://github.com/apache/spark/pull/29788#issuecomment-695530700
Since there was another PR in the same area committed that broke the
existing integration tests in this area I don't feel confident with my soft
reservations and switching to a
holdenk commented on pull request #29788:
URL: https://github.com/apache/spark/pull/29788#issuecomment-694327672
Please tag this PR as WIP until it is tested, thanks for working on
improving the code though @Ngone51 :)