Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user stanzhai commented on the issue:
https://github.com/apache/spark/pull/18986
@gatorsmile @DonnyZone When comparing a string to a int in Hive, it will
cast string type to double.
```
hive> select * from tb;
0 0
0.1 0
true0
Github user stanzhai commented on the issue:
https://github.com/apache/spark/pull/18986
@DonnyZone @gatorsmile @cloud-fan PostgreSQL will throw an error when
comparing a string to a int.
```
postgres=# select * from tb;
a | b
--+---
0.1 | 1
a
Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/18986
@gatorsmile For this issue, I think the behevior in PromoteStrings rule is
reasonable, but there are problems in underlying converter UTF8String.
As described in PR-15880
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18986
Yea, since this topic is important for some users, I mean we better move
the doc into `./docs/` ( I feel novices dont seem to check the code documents).
---
If your project is set up for it, you
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18986
@maropu Previously I did it in
https://github.com/apache/spark/blob/master/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercionSuite.scala#L34-L59
We can
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18986
Either way, we couldn't touch on this behaviour until the next major
release? (It seems we've already fixed this policy for the 2.x releases in
#15880... maybe). Btw, I feel we first should document
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18986
In Hive, `1 = 'true'` will return `true`? `19157170390056973L =
'19157170390056971'` will return `true`?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/18986
@gatorsmile Seems cast to `double` is correct.
```
hive> create table spark_21646(c1 string, c2 string);
hive> insert into spark_21646
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18986
@stanzhai @wangyum How about Hive? I think your usage scenarios are for
hive compatibility, right?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/18986
There is a issue if a string value out of double range, see:
[SPARK-21646](https://issues.apache.org/jira/browse/SPARK-21646).
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80868/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18986
**[Test build #80868 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80868/testReport)**
for PR 18986 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80866/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18986
**[Test build #80866 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80866/testReport)**
for PR 18986 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18986
**[Test build #80868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80868/testReport)**
for PR 18986 at commit
Github user stanzhai commented on the issue:
https://github.com/apache/spark/pull/18986
In MySQL conversion of values from one string type to numeric, will be
compared as floating-point (real) numbers.
[](https://dev.mysql.com/doc/refman/5.7/en/type-conversion.html)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18986
**[Test build #80866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80866/testReport)**
for PR 18986 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80849/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18986
**[Test build #80849 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80849/testReport)**
for PR 18986 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18986
@stanzhai Implicit type casting has to follow some rules. If you think this
is a bug, could you do a complete investigation about this issue? For example,
how Hive handles the type promotion?
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18986
**[Test build #80849 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80849/testReport)**
for PR 18986 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18986
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18986
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
30 matches
Mail list logo