Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85848236
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85847230
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15695
Sure.
---
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
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85849098
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85847599
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15675
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67833/
Test PASSed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85850400
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/util/ByteArrayReadableChannel.java
---
@@ -0,0 +1,71 @@
+/*
+ * Licensed to
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85849036
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85849245
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85851023
--- Diff: docs/configuration.md ---
@@ -1435,6 +1435,21 @@ Apart from these, the following properties are also
available, and may be useful
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85845790
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/SaslClientBootstrap.java
---
@@ -88,9 +90,25 @@ public void
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85850499
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85848147
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache
Github user zsxwing closed the pull request at:
https://github.com/apache/spark/pull/15695
---
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
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85849596
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesConfigMessage.java
---
@@ -0,0 +1,104 @@
+/*
+ * Licensed to the
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85849759
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesConfigMessage.java
---
@@ -0,0 +1,104 @@
+/*
+ * Licensed to the
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/15699
LGTM
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15701
**[Test build #67836 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67836/consoleFull)**
for PR 15701 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15696
**[Test build #67840 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67840/consoleFull)**
for PR 15696 at commit
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/15701
merging to master. Thanks
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15677
@HyukjinKwon Sure, I can try it.
Actually, we might need to find multiple typical function APIs as examples
to discuss what are the best way to document/specify the argument/parameter
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15705
**[Test build #67846 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67846/consoleFull)**
for PR 15705 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15651
---
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
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15666#discussion_r85830311
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1700,19 +1700,34 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15699
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15699
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67830/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15696
**[Test build #67840 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67840/consoleFull)**
for PR 15696 at commit
GitHub user liancheng opened a pull request:
https://github.com/apache/spark/pull/15703
[SPARK-18186] Migrate HiveUDAFFunction to TypedImperativeAggregate for
partial aggregation support
## What changes were proposed in this pull request?
This PR migrates
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67837 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67837/consoleFull)**
for PR 1 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67837/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15688
**[Test build #67847 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67847/consoleFull)**
for PR 15688 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13575
@lw-lin sorry we haven't visited this pr in the past, but the goal of this
pr will be accomplished by https://issues.apache.org/jira/browse/SPARK-17924
(as a side effect). It reduces a lot of
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15706
**[Test build #3385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3385/consoleFull)**
for PR 15706 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67869/
Test FAILed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15709
I see. That seems nice too but I hope the version is fixed if this one is
also fine too..
I am worried if the version is suddenly bumped up and some tests are failed
(at least, I was a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67869/consoleFull)**
for PR 1 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67850/consoleFull)**
for PR 15693 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15675
**[Test build #67851 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67851/consoleFull)**
for PR 15675 at commit
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15696#discussion_r85860514
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileCommitProtocol.scala
---
@@ -0,0 +1,223 @@
+/*
+ * Licensed to the
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15696#discussion_r85860719
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/WriteOutput.scala
---
@@ -218,22 +210,24 @@ object WriteOutput extends Logging {
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15696#discussion_r85859487
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileCommitProtocol.scala
---
@@ -0,0 +1,223 @@
+/*
+ * Licensed to the
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15696#discussion_r85859041
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileCommitProtocol.scala
---
@@ -0,0 +1,223 @@
+/*
+ * Licensed to the
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15696#discussion_r85859292
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileCommitProtocol.scala
---
@@ -0,0 +1,223 @@
+/*
+ * Licensed to the
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15696#discussion_r85859055
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileCommitProtocol.scala
---
@@ -0,0 +1,223 @@
+/*
+ * Licensed to the
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15706
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 sethah commented on the issue:
https://github.com/apache/spark/pull/15593
ping @MLnick @jkbradley This should be a nice performance boost for MLOR in
ML, hopefully we can get it in for 2.1. If you get some time to review or run
tests I'd really appreciate it.
---
If
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15704
**[Test build #67857 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67857/consoleFull)**
for PR 15704 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15697
Build started: [SparkR] `ALL`
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15677
Thank you for your work!
I might offend you in my previous comments. That's certainly not my
intention. Previously, I worked for the infrastructure software on mainframe.
IMO, Spark is
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15667
Let's do it in a follow-up.
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15696
Closing this in favor of https://github.com/apache/spark/pull/15707
---
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 rxin closed the pull request at:
https://github.com/apache/spark/pull/15696
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15694
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15707
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15707
**[Test build #67855 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67855/consoleFull)**
for PR 15707 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15694
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67856/
Test PASSed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15673
LGTM
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15707
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67855/
Test PASSed.
---
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15707
This LGTM, just a minor comment
---
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 zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/15669#discussion_r85872343
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1716,29 +1716,12 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15669
**[Test build #67868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67868/consoleFull)**
for PR 15669 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15688
@gatorsmile that's a good point. I checked with hive, the behaviour is:
1. if the given partition spec specifies all partition columns, throw
exception if the partition not exist
2.
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/13599
Thanks for the review @mridulm , this approach is trying the move the
overhead from user to cluster. User just need to specify the requirement file
and spark will set up the virtualenv
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15710
**[Test build #67873 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67873/consoleFull)**
for PR 15710 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15302
Thank you, @hvanhovell !
---
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 sethah commented on the issue:
https://github.com/apache/spark/pull/15414
LGTM as well after adding @jkbradley's suggestion.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15704
**[Test build #67843 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67843/consoleFull)**
for PR 15704 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15704
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67843/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15704
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/15706
**[Test build #3383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3383/consoleFull)**
for PR 15706 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15706
LGTM pending Jenkins.
---
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 rxin opened a pull request:
https://github.com/apache/spark/pull/15707
[SPARK-18024][SQL] Introduce an internal commit protocol API - rebased
## What changes were proposed in this pull request?
This patch introduces an internal commit protocol API that is used by the
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15703
cc @tejasapatil
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15707
This is the same as https://github.com/apache/spark/pull/15696
but rebased with https://github.com/apache/spark/pull/15633
---
If your project is set up for it, you can reply to this email
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15667
lgtm if tests pass
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15414
**[Test build #67861 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67861/consoleFull)**
for PR 15414 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15688
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67847/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15688
**[Test build #67847 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67847/consoleFull)**
for PR 15688 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15705
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67848/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15633
**[Test build #3381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3381/consoleFull)**
for PR 15633 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15705
**[Test build #67848 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67848/consoleFull)**
for PR 15705 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15688
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
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15707
This lgtm, modulo the comments in #15696
---
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/15706
**[Test build #3383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3383/consoleFull)**
for PR 15706 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15705
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/15675
**[Test build #67851 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67851/consoleFull)**
for PR 15675 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15675
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67851/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67860/
Test PASSed.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15673
@mallman can you bring this up-to-date?
---
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 cjjnjust commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85875690
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/SaslRpcHandler.java
---
@@ -80,46 +84,71 @@ public void
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/15710
[SPARK-18025] Use commit protocol API in structured streaming
## What changes were proposed in this pull request?
This patch adds a new commit protocol implementation
ManifestFileCommitProtocol
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/15709
This approach is fine - The other thing we could do is just use the latest
stable version as described in https://cloud.r-project.org/bin/windows/base/ -
If you see the link at the bottom it says
GitHub user seyfe opened a pull request:
https://github.com/apache/spark/pull/15706
[SPARK-18189] [Core] Fix serialization issue in KeyValueGroupedDataset
## What changes were proposed in this pull request?
Likewise
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15696
**[Test build #67852 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67852/consoleFull)**
for PR 15696 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15702
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/15702
**[Test build #67839 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67839/consoleFull)**
for PR 15702 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15633
LGTM, pending jenkins
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67839/
Test FAILed.
---
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15673#discussion_r85856641
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -585,7 +586,31 @@ private[client] class Shim_v0_13 extends Shim_v0_12
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15692
@jkbradley I add this param for GBTClassifier and GBTRegressor.
---
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
101 - 200 of 572 matches
Mail list logo
