dongjoon-hyun commented on pull request #30379:
URL: https://github.com/apache/spark/pull/30379#issuecomment-727329518
Since the last commit is only for commenting and this PR already passed, I
merged this. Thanks~
This is
dongjoon-hyun commented on pull request #30379:
URL: https://github.com/apache/spark/pull/30379#issuecomment-727282415
Your idea is better and correct because there is no conf yet~
> Yea, but, on second thought, merging this fist looks fine, too.
dongjoon-hyun commented on pull request #30379:
URL: https://github.com/apache/spark/pull/30379#issuecomment-727280631
@maropu That's a good point. Initially, I thought three steps. 1) we add a
benchmark with the baseline first, 2) merge the #30341, 3) update this
benchmark in another PR.
dongjoon-hyun commented on pull request #30379:
URL: https://github.com/apache/spark/pull/30379#issuecomment-727271068
Thank you so much for this additional work, @viirya !
This is an automated message from the Apache Git