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:
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
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
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
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):
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
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
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
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
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
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
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):
16 matches
Mail list logo
