Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111679144
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -92,6 +92,7 @@ abstract class Optimizer(sessionCatalog:
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111678668
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/objects.scala
---
@@ -96,3 +98,23 @@ object CombineTypedFilters extends Rule[Lo
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111678646
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/objects.scala
---
@@ -96,3 +99,23 @@ object CombineTypedFilters extends Rule[Lo
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17641#discussion_r111677733
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -34,6 +33,9 @@ import org.apache.spark.u
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17635
---
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 the feature is ena
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17635
LGTM, merging to master! @gatorsmile shall we backport this PR?
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17568
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75831/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17568
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17568
**[Test build #75831 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75831/testReport)**
for PR 17568 at commit
[`ea654bf`](https://github.com/apache/spark/commit/e
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111677168
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -92,6 +92,7 @@ abstract class Optimizer(sessionCatal
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111677163
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/objects.scala
---
@@ -96,3 +98,23 @@ object CombineTypedFilters extends Rul
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17639
---
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 the feature is ena
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111677150
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -368,6 +369,8 @@ case class NullPropagation(conf:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17639
merging to master!
---
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,
Github user yanji84 commented on the issue:
https://github.com/apache/spark/pull/17109
@srowen @tnachen confirm merge
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17645
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17645
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75830/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17645
**[Test build #75830 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75830/testReport)**
for PR 17645 at commit
[`6541f69`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17568
**[Test build #75831 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75831/testReport)**
for PR 17568 at commit
[`ea654bf`](https://github.com/apache/spark/commit/ea
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17645
**[Test build #75830 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75830/testReport)**
for PR 17645 at commit
[`6541f69`](https://github.com/apache/spark/commit/65
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17644
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75829/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17644
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17644
**[Test build #75829 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75829/testReport)**
for PR 17644 at commit
[`0348a96`](https://github.com/apache/spark/commit/0
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/17645
[SPARK-20348] [ML] Support squared hinge loss (L2 loss) for LinearSVC
## What changes were proposed in this pull request?
While Hinge loss is the standard loss function for linear SVM, Squar
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17640
What's the number you use for this big number?
---
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 doe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17644
**[Test build #75829 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75829/testReport)**
for PR 17644 at commit
[`0348a96`](https://github.com/apache/spark/commit/03
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/17644
[SPARK-17729] [SQL] Enable creating hive bucketed tables
## What changes were proposed in this pull request?
Hive allows inserting data to bucketed table without guaranteeing bucketed
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/15300
since trunk had diverged a lot since this PR was created, closed this and
created a fresh one at : https://github.com/apache/spark/pull/17644
---
If your project is set up for it, you can reply
Github user tejasapatil closed the pull request at:
https://github.com/apache/spark/pull/15300
---
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 the feature i
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/17590
I think @HyukjinKwon interpretation is good !
---
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 feat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17568
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75825/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17568
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17568
**[Test build #75825 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75825/testReport)**
for PR 17568 at commit
[`1d98a19`](https://github.com/apache/spark/commit/1
Github user yanji84 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17109#discussion_r111671944
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtil.scala
---
@@ -99,6 +99,26 @@ private[me
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17109
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75828/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17109
**[Test build #75828 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75828/testReport)**
for PR 17109 at commit
[`2f3f8b2`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17109
Merged build finished. Test PASSed.
---
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
e
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/17640
If I use very big number, then sparkR shell will get the following output:
> collect(df1)
a b cd
1 Inf 1 1 Inf
So the overflow prob
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17109
**[Test build #75828 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75828/testReport)**
for PR 17109 at commit
[`2f3f8b2`](https://github.com/apache/spark/commit/2f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17643
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17643
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75826/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75827/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17643
**[Test build #75826 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75826/testReport)**
for PR 17643 at commit
[`cd65ace`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75827/testReport)**
for PR 13440 at commit
[`5cbb21c`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111671500
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/objects.scala
---
@@ -96,3 +99,23 @@ object CombineTypedFilters extends Ru
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111671495
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/objects.scala
---
@@ -96,3 +99,23 @@ object CombineTypedFilters extends Ru
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75827/testReport)**
for PR 13440 at commit
[`5cbb21c`](https://github.com/apache/spark/commit/5c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17643
**[Test build #75826 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75826/testReport)**
for PR 17643 at commit
[`cd65ace`](https://github.com/apache/spark/commit/cd
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/17643
[SPARK-20345] Fix STS error handling logic on HiveSQLException
## What changes were proposed in this pull request?
[SPARK-5100](https://github.com/apache/spark/commit/343d3bfafd449a0
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111670773
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -437,11 +437,12 @@ object MapObjects {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111670759
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystTypeConverters.scala
---
@@ -40,7 +40,7 @@ object CatalystTypeConverters {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111670727
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DatasetPrimitiveSuite.scala ---
@@ -20,7 +20,10 @@ package org.apache.spark.sql
import scala.
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17568#discussion_r111670721
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DatasetPrimitiveSuite.scala ---
@@ -262,5 +265,4 @@ class DatasetPrimitiveSuite extends QueryTest
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17635
+1
---
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 the f
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17635
cc @cloud-fan
---
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, o
Github user tnachen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17109#discussion_r111670608
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtil.scala
---
@@ -99,6 +99,26 @@ private[me
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17635
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75824/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17635
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17635
**[Test build #75824 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75824/testReport)**
for PR 17635 at commit
[`508a43d`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17568
**[Test build #75825 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75825/testReport)**
for PR 17568 at commit
[`1d98a19`](https://github.com/apache/spark/commit/1d
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17623
What are the external impacts of these changes? Which commands are
impacted?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17635
**[Test build #75824 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75824/testReport)**
for PR 17635 at commit
[`508a43d`](https://github.com/apache/spark/commit/50
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17149
(I think I should cc @srowen too FYI because he reviewed all my PRs fixing
the tests on Windows. The point here seems to me okay to use URIs in general.)
---
If your project is set up for it, y
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17642
Thank you for your approval @srowen.
---
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
en
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17642
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75823/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17642
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17642
**[Test build #75823 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75823/testReport)**
for PR 17642 at commit
[`a983010`](https://github.com/apache/spark/commit/a
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17109
Is this one still good to go? I see a late comment above. I can merge it,
no problem.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17416
Thanks @BryanCutler ! so, eh, do you mean you have a working version of
this change? feel free to open it as a PR to master or to this PR or whatever,
whatever's simplest. I'd be pleased to have a lo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17642
**[Test build #75823 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75823/testReport)**
for PR 17642 at commit
[`a983010`](https://github.com/apache/spark/commit/a9
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17642
Sure, I will!
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17628
---
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 the feature is ena
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17628
Merged to master
---
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 i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17568
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17642
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75821/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75822/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17642
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17568
**[Test build #75822 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75822/testReport)**
for PR 17568 at commit
[`efc714b`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17642
**[Test build #75821 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75821/testReport)**
for PR 17642 at commit
[`1ae57f2`](https://github.com/apache/spark/commit/1
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17642
Before doing this, can you instead try adding `org.apache.avro:avro` as a
direct dependency of `core`? it looks like it should be, and, could be the
reason the resolution is not as expected in this c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17568
**[Test build #75822 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75822/testReport)**
for PR 17568 at commit
[`efc714b`](https://github.com/apache/spark/commit/ef
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17642
**[Test build #75821 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75821/testReport)**
for PR 17642 at commit
[`1ae57f2`](https://github.com/apache/spark/commit/1a
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17642
retest this please
---
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 s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17642
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75820/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17642
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
e
86 matches
Mail list logo
