Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22407
@cloud-fan @ueshin Thank you very much !!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22407
thanks, merging to master/2.4!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96475/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96475/testReport)**
for PR 22407 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96475 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96475/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22407
LGTM except one comment
---
-
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/22407
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/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96421/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96421 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96421/testReport)**
for PR 22407 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22407
LGTM, pending Jenkins.
---
-
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/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
**[Test build #96421 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96421/testReport)**
for PR 22407 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22407
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22407
So, before #22448 (branch-2.4 if we don't backport #22448), we don't coerce
between decimals, and after that (master, not merged yet though), we will do,
right?
I'm okay with the behavior.
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22407
@ueshin Wenchen thought it may be risky to backport the fix to
tighestCommonType. Given this, can this be looked at now ?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96027/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
**[Test build #96027 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96027/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96028/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96028 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96028/testReport)**
for PR 22407 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96028 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96028/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22407
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96027 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96027/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96024/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96024 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96024/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
**[Test build #96024 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96024/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22407
@dilipbiswal Thanks! I'll take a look at it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22407
> Seems like ArrayContains, ArrayRemove, and ElementAt for map types could
have the same problem?
@ueshin I have a PR opened for ArrayContains
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22407
cc @ueshin
---
-
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/22407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96014/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
**[Test build #96014 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96014/testReport)**
for PR 22407 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
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/22407
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/22407
**[Test build #96014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96014/testReport)**
for PR 22407 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22407
retest this please
---
-
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/22407
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22407
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96008/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22407
**[Test build #96008 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96008/testReport)**
for PR 22407 at commit
49 matches
Mail list logo