Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19821
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19821
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84293/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19821
**[Test build #84293 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84293/testReport)**
for PR 19821 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19821
LGTM, can you remove `WIP` in PR title?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19821
**[Test build #84293 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84293/testReport)**
for PR 19821 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19821
Sure, I have resolved the conflict in my environment. I will commit soon.
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19821
@kiszk Can you fix the conflict? now we can add a middle-advanced version:
```
def splitExpressions(
expressions: Seq[String],
funcName: String,
extraArguments:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19821
I have no strong preference.
@gatorsmile WDYT?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19821
is it really worth? seems not used in many places and eventually the
if-else will be removed after we make `splitExpression` work with whole stage
codegen
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19821
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19821
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84191/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19821
**[Test build #84191 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84191/testReport)**
for PR 19821 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19821
**[Test build #84191 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84191/testReport)**
for PR 19821 at commit
13 matches
Mail list logo