Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21482
ping @NihalHarish
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21482
@NihalHarish shall we leave this closed for now if you don't have any
opinion on ^?
---
-
To unsubscribe, e-mail: reviews-un
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21482
OK I double checked. I don't think we should be adding this functionality,
since different databases implemented it differently, and it is somewhat
difficult to create Infinity in Spark SQL given we re
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21482
Hey I have an additional thought on this. Will leave it in the next ten
mins.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92330/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
**[Test build #92330 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92330/testReport)**
for PR 21482 at commit
[`d60aa21`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92330 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92330/testReport)**
for PR 21482 at commit
[`d60aa21`](https://github.com/apache/spark/commit/d6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92315/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92315/testReport)**
for PR 21482 at commit
[`cb8f9d0`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92315/testReport)**
for PR 21482 at commit
[`cb8f9d0`](https://github.com/apache/spark/commit/cb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92307/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92307 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92307/testReport)**
for PR 21482 at commit
[`be23549`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92307 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92307/testReport)**
for PR 21482 at commit
[`be23549`](https://github.com/apache/spark/commit/be
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92262/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
**[Test build #92262 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92262/testReport)**
for PR 21482 at commit
[`6bd6735`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92262 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92262/testReport)**
for PR 21482 at commit
[`6bd6735`](https://github.com/apache/spark/commit/6b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92224/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
**[Test build #92224 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92224/testReport)**
for PR 21482 at commit
[`b727838`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #92224 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92224/testReport)**
for PR 21482 at commit
[`b727838`](https://github.com/apache/spark/commit/b7
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21482
I have no more comments except the one above.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/21482
Any further comments here?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91601/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91601 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91601/testReport)**
for PR 21482 at commit
[`559900a`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91601 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91601/testReport)**
for PR 21482 at commit
[`559900a`](https://github.com/apache/spark/commit/55
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21482
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91575/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
**[Test build #91575 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91575/testReport)**
for PR 21482 at commit
[`559900a`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91575 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91575/testReport)**
for PR 21482 at commit
[`559900a`](https://github.com/apache/spark/commit/55
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/21482
I think consistency in Spark's naming convention (and therefore increased
discoverability by users) outweighs the advantage of naming it exactly for
the Impala equivalent. I do agree that multi
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21482
Thanks, Henry. In general I'm not a huge fan of adding something because
hypothetically somebody might want it. Also if you want this to be compatible
with Impala, wouldn't you want to name this the sa
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/21482
@rxin, that in itself is a bit weird, but there are ways to express inf
values in Scala and thus inf values can show up flowing through Spark plans.
I'm not sure MySQL has any such facility.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91518/testReport)**
for PR 21482 at commit
[`6a4d46e`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91517 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91517/testReport)**
for PR 21482 at commit
[`f240fdf`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91518/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91517/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91518/testReport)**
for PR 21482 at commit
[`6a4d46e`](https://github.com/apache/spark/commit/6a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91517 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91517/testReport)**
for PR 21482 at commit
[`f240fdf`](https://github.com/apache/spark/commit/f2
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21482
@henryr 1.0/0.0 also returns null in Spark SQL ...
```
scala> sql("select cast(1.0 as double)/cast(0 as double)").show()
+-+
|(CAST(1.0 AS DOU
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/21482
@rxin Other engines are all over the place:
* MySQL doesn't have support for infinity (based on my cursory look) - 1.0
/ 0.0 is written as `null`. Also seems to be true of SQLite.
* Postg
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91494/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
**[Test build #91494 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91494/testReport)**
for PR 21482 at commit
[`663fa47`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91494 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91494/testReport)**
for PR 21482 at commit
[`663fa47`](https://github.com/apache/spark/commit/66
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21482
How is this done in other databases? I don't think we want to invent new
ways on these basic primitives.
---
-
To unsubscribe, e-ma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91467/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91467 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91467/testReport)**
for PR 21482 at commit
[`432c61b`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91467 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91467/testReport)**
for PR 21482 at commit
[`432c61b`](https://github.com/apache/spark/commit/43
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91462/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91462/testReport)**
for PR 21482 at commit
[`d381f0c`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91462 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91462/testReport)**
for PR 21482 at commit
[`d381f0c`](https://github.com/apache/spark/commit/d3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91459/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91459/testReport)**
for PR 21482 at commit
[`13b5aaa`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91459 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91459/testReport)**
for PR 21482 at commit
[`13b5aaa`](https://github.com/apache/spark/commit/13
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Merged build finished. Test FAILed.
---
-
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/21482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91423/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91423 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91423/testReport)**
for PR 21482 at commit
[`7e396f7`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91423 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91423/testReport)**
for PR 21482 at commit
[`7e396f7`](https://github.com/apache/spark/commit/7e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91417/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
**[Test build #91417 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91417/testReport)**
for PR 21482 at commit
[`069f9d9`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91418/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91418/testReport)**
for PR 21482 at commit
[`f34bfdc`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91418/testReport)**
for PR 21482 at commit
[`f34bfdc`](https://github.com/apache/spark/commit/f3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91417 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91417/testReport)**
for PR 21482 at commit
[`069f9d9`](https://github.com/apache/spark/commit/06
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91408/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91408/testReport)**
for PR 21482 at commit
[`9ab0eb2`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91405/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91405/testReport)**
for PR 21482 at commit
[`bcdaab2`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91408/testReport)**
for PR 21482 at commit
[`9ab0eb2`](https://github.com/apache/spark/commit/9a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91405/testReport)**
for PR 21482 at commit
[`bcdaab2`](https://github.com/apache/spark/commit/bc
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21482
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21482
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21482
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/21482
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
90 matches
Mail list logo
