[GitHub] [spark] dongjoon-hyun commented on pull request #30341: [SPARK-33427][SQL] Add subexpression elimination for interpreted expression evaluation

2020-11-17 Thread GitBox
dongjoon-hyun commented on pull request #30341: URL: https://github.com/apache/spark/pull/30341#issuecomment-729051922 Thank you, @viirya , @maropu , @cloud-fan ! This is an automated message from the Apache Git Service. To

[GitHub] [spark] dongjoon-hyun commented on pull request #30341: [SPARK-33427][SQL] Add subexpression elimination for interpreted expression evaluation

2020-11-14 Thread GitBox
dongjoon-hyun commented on pull request #30341: URL: https://github.com/apache/spark/pull/30341#issuecomment-727420586 No need to hurry~ Take your time. I just want to give you feedback swiftly from my side. This is an

[GitHub] [spark] dongjoon-hyun commented on pull request #30341: [SPARK-33427][SQL] Add subexpression elimination for interpreted expression evaluation

2020-11-14 Thread GitBox
dongjoon-hyun commented on pull request #30341: URL: https://github.com/apache/spark/pull/30341#issuecomment-727331477 #30379 is merged. Could you rebased to the master and regenerate the result, @viirya . Sure, that's the exact reason I asked their opinion once more.  > Let's

[GitHub] [spark] dongjoon-hyun commented on pull request #30341: [SPARK-33427][SQL] Add subexpression elimination for interpreted expression evaluation

2020-11-14 Thread GitBox
dongjoon-hyun commented on pull request #30341: URL: https://github.com/apache/spark/pull/30341#issuecomment-727254222 It would be great if we can have a benchmark suite (from the code in the PR description), @viirya . This

[GitHub] [spark] dongjoon-hyun commented on pull request #30341: [SPARK-33427][SQL] Add subexpression elimination for interpreted expression evaluation

2020-11-11 Thread GitBox
dongjoon-hyun commented on pull request #30341: URL: https://github.com/apache/spark/pull/30341#issuecomment-725860167 Thank you for pinging me, @viirya . This is an automated message from the Apache Git Service. To respond