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 th
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 S
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 S
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 en
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 A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85848802
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache S
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85848940
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache S
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85847613
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/AesCipher.java
---
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the Apache S
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85847089
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/SaslRpcHandler.java
---
@@ -80,46 +84,71 @@ public void receive(TransportClient c
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15699
Thanks! Merging to master and 2.0.
---
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
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85850906
--- Diff:
common/network-common/src/test/java/org/apache/spark/network/sasl/SparkSaslSuite.java
---
@@ -374,6 +375,69 @@ public void testDelegates() throws E
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r85846395
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/SaslRpcHandler.java
---
@@ -80,46 +84,71 @@ public void receive(TransportClient c
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 feat
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15302
Hi, @hvanhovell .
I made another attempt #15704 by using 'Expression' as you commented.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15675
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/15699
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15696
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67838/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15696
**[Test build #67838 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67838/consoleFull)**
for PR 15696 at commit
[`040bbba`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15696
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 wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15697
windows failure seems unrelated
---
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
enable
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/15669
that's correct, it is due to `spark.files`, jira has been updated. Will
update the PR soon.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
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
[`767fef8`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15701
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/15701
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67836/
Test PASSed.
---
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
t
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
[`51d0919`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15696
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67840/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15696
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/15703
**[Test build #67844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67844/consoleFull)**
for PR 15703 at commit
[`5a23a97`](https://github.com/apache/spark/commit/5
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15302
@dongjoon-hyun I'll take a look tomorrow.
---
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 featur
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15701
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15626
**[Test build #67845 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67845/consoleFull)**
for PR 15626 at commit
[`8ee336d`](https://github.com/apache/spark/commit/8
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
[`b397d04`](https://github.com/apache/spark/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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15677
Alright, then will try to get rid of the arguments part. Thank you all very
much sincerely and I apologise the noise I caused. @gatorsmile I will keep in
mind you comments too.
---
If your pro
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
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 ericl opened a pull request:
https://github.com/apache/spark/pull/15705
[SPARK-18183] [SPARK-18184] Fix INSERT [INTO|OVERWRITE] TABLE ... PARTITION
for Datasource tables
## What changes were proposed in this pull request?
There are a couple issues with the curre
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15705
cc @cloud-fan @yhuai
---
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 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
[`669e6cc`](https://github.com/apache/spark/commit/6
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15688
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 so,
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
[`a9eec51`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15705
**[Test build #67846 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67846/consoleFull)**
for PR 15705 at commit
[`669e6cc`](https://github.com/apache/spark/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
e
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/67846/
Test FAILed.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15703
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/15703
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67842/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15703
**[Test build #67842 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67842/consoleFull)**
for PR 15703 at commit
[`c0029f1`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15633
**[Test build #3381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3381/consoleFull)**
for PR 15633 at commit
[`4d96725`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15673
**[Test build #3382 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3382/consoleFull)**
for PR 15673 at commit
[`4c438c8`](https://github.com/apache/spark/commit/
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
[DataSet.scala](https://github.com/apache/spark/blob/master/sql
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15706
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 feat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15705
**[Test build #67848 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67848/consoleFull)**
for PR 15705 at commit
[`fec7c9e`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15705
**[Test build #67849 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67849/consoleFull)**
for PR 15705 at commit
[`07c6787`](https://github.com/apache/spark/commit/0
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15702#discussion_r85859357
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/CalendarInterval.java
---
@@ -252,6 +252,10 @@ public static long parseSecondNano(String se
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15702#discussion_r85859683
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -536,6 +535,37 @@ class Dataset[T] private[sql](
}
/**
+ *
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15705#discussion_r85859910
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -345,18 +346,32 @@ case class Broadca
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14803#discussion_r85860069
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -608,6 +614,81 @@ class FileStreamSourceSuite extends
Fi
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
[`5006bcb`](https://github.com/apache/spark/commit/5
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
[`4b29d8f`](https://github.com/apache/spark/commit/4
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
[`84f2315`](https://github.com/apache/spark/commit/
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 f
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
e
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 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
[`5b92132`](https://github.com/apache/spark/commit/
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 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 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
[`cd23d2f`](https://github.com/apache/spark/commit/c
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
e
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/11105#discussion_r85860913
--- Diff:
core/src/test/scala/org/apache/spark/DataPropertyAccumulatorSuite.scala ---
@@ -0,0 +1,361 @@
+/*
+ * Licensed to the Apache Software Founda
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15703
**[Test build #67844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67844/consoleFull)**
for PR 15703 at commit
[`5a23a97`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15703
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/15703
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67844/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15626
**[Test build #67831 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67831/consoleFull)**
for PR 15626 at commit
[`d6fec94`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15626
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67831/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15626
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 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 Ap
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 Ap
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 Ap
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 you
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 Ap
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 Ap
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 marmbrus commented on the issue:
https://github.com/apache/spark/pull/15702
@ekl - flaky test... Should we turn it off for now?
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 y
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15706
This guy @transient must get a lot of emails ...
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15706
Can you change the title to "[SQL]" rather than "[core]"?
---
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 t
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15667#discussion_r85861722
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -257,7 +258,31 @@ case class InsertIntoHiveTable(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15667#discussion_r85861794
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -257,7 +258,31 @@ case class InsertIntoHiveTable(
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 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
[`07160b9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15667
**[Test build #67853 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67853/consoleFull)**
for PR 15667 at commit
[`bd22150`](https://github.com/apache/spark/commit/b
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15702
I'm still trying to find a failure that includes
https://github.com/apache/spark/pull/15701/files. Until then it's hard to debug.
---
If your project is set up for it, you can reply to this email and
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 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 and
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 ericl opened a pull request:
https://github.com/apache/spark/pull/15708
[SPARK-18167] [SQL] Retry when the SQLQuerySuite test flakes
## What changes were proposed in this pull request?
This will re-run the flaky test a few times after it fails. This will help
de
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15707
**[Test build #67855 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67855/consoleFull)**
for PR 15707 at commit
[`0647959`](https://github.com/apache/spark/commit/0
301 - 400 of 601 matches
Mail list logo