Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23079
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user aokolnychyi commented on the issue:
https://github.com/apache/spark/pull/23079
LGTM as well.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98996/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23079
**[Test build #98996 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98996/testReport)**
for PR 23079 at commit
[`6646a96`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5136/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23079
**[Test build #98996 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98996/testReport)**
for PR 23079 at commit
[`6646a96`](https://github.com/apache/spark/commit/66
Github user aokolnychyi commented on the issue:
https://github.com/apache/spark/pull/23079
@rednaxelafx I am glad the rule gets more adoption. Renaming also makes
sense to me.
Shall we extend `ReplaceNullWithFalseEndToEndSuite` as well?
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98975/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23079
**[Test build #98975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98975/testReport)**
for PR 23079 at commit
[`710c886`](https://github.com/apache/spark/commit/7
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/23079
cc @aokolnychyi : I'd like to propose renaming the rule you introduced to
add a `-InPredicate` suffix, because obviously we can't replace arbitrary
`null`s with `false`, but only the ones that a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23079
**[Test build #98975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98975/testReport)**
for PR 23079 at commit
[`710c886`](https://github.com/apache/spark/commit/71
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5118/
16 matches
Mail list logo