Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23254
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/23254
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23254
**[Test build #99826 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99826/testReport)**
for PR 23254 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23239
**[Test build #99820 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99820/testReport)**
for PR 23239 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23239
**[Test build #99819 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99819/testReport)**
for PR 23239 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23254
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99826/
Test FAILed.
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23249#discussion_r239754619
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
---
@@ -118,10 +115,12 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23207
**[Test build #99825 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99825/testReport)**
for PR 23207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23207
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23249
**[Test build #99821 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99821/testReport)**
for PR 23249 at commit
Github user seancxmao commented on the issue:
https://github.com/apache/spark/pull/23224
@felixcheung Yes, that makes sense. I have added a commit to check that.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23224
**[Test build #99829 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99829/testReport)**
for PR 23224 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23254
**[Test build #99826 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99826/testReport)**
for PR 23254 at commit
Github user httfighter commented on the issue:
https://github.com/apache/spark/pull/22683
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/22683
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23253
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99824/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23253
**[Test build #99824 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99824/testReport)**
for PR 23253 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23245
**[Test build #99818 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99818/testReport)**
for PR 23245 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99817/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23156
**[Test build #99822 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99822/testReport)**
for PR 23156 at commit
GitHub user gaborgsomogyi opened a pull request:
https://github.com/apache/spark/pull/23254
[SPARK-26304][SS] Add default value to
spark.kafka.sasl.kerberos.service.name parameter
## What changes were proposed in this pull request?
spark.kafka.sasl.kerberos.service.name is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23253
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23239
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99816/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23253
**[Test build #99828 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99828/testReport)**
for PR 23253 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23207
**[Test build #99817 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99817/testReport)**
for PR 23207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23254
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/22707
cc @wangyum
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23253
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23239
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/22952
BTW: [HADOOP-15748](https://issues.apache.org/jira/browse/HADOOP-15748),
*S3 listing inconsistency can raise NPE in globber*
Could be backported to 2.8+; low risk
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23239
**[Test build #99816 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99816/testReport)**
for PR 23239 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23253
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23207
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/23225
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/22952
> Looks like we can leverage GlobPattern but it is marked as @Private.
I will happily switch this from private to public/evolving if you submit a
patch against hadoop-trunk; backport
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23253
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23239
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99820/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23239
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23239
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23218
I'm not sure. I can't find any other reference to this crash and 2.12.8. It
could be something only Spark happens to trigger, or could be specific to this
JVM + platform but not Spark or Scala. We
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23156
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99822/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23156
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22683
**[Test build #99827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99827/testReport)**
for PR 22683 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99818/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23249
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99821/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23249
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23207
**[Test build #99825 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99825/testReport)**
for PR 23207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23242
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23224#discussion_r239837818
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/metric/SQLMetricsSuite.scala
---
@@ -80,8 +80,10 @@ class SQLMetricsSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23245
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23224
**[Test build #99829 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99829/testReport)**
for PR 23224 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22273
How about if we checked an env variable like EXPECT_ARROW (and set it when
tests are running in Jenkins) and if it's set but we don't have the required
versions installed fail? That way we know if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23252
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23252
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23208#discussion_r239888975
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/SupportsBatchWrite.java
---
@@ -25,14 +25,14 @@
import
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23208#discussion_r239888795
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/TableProvider.java ---
@@ -25,7 +25,10 @@
* The base interface for v2 data sources
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23208#discussion_r239889152
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,52 +17,49 @@
package
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/23254
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23242
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99825/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23242
**[Test build #99831 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99831/testReport)**
for PR 23242 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23242
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99831/
Test FAILed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20146
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/23226
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23254
**[Test build #99830 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99830/testReport)**
for PR 23254 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23253
**[Test build #99828 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99828/testReport)**
for PR 23253 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/23225#discussion_r239872296
--- Diff:
core/src/test/java/org/apache/spark/shuffle/sort/UnsafeShuffleWriterSuite.java
---
@@ -235,6 +235,7 @@ public void writeEmptyIterator()
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22957
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22957
@cloud-fan @gatorsmile I updated the PR according to the previous
suggestions and added a new dedicated test suite.
May you please review this again? Thanks.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22957
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23224
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99829/
Test PASSed.
---
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/23252
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22683
**[Test build #99827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99827/testReport)**
for PR 22683 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20146
**[Test build #99832 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99832/testReport)**
for PR 20146 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #99833 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99833/testReport)**
for PR 22952 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23245
Thank you all. Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23224
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23252
**[Test build #99835 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99835/testReport)**
for PR 23252 at commit
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/23252#discussion_r239884995
--- Diff: core/src/test/scala/org/apache/spark/SecurityManagerSuite.scala
---
@@ -440,12 +473,27 @@ class SecurityManagerSuite extends SparkFunSuite with
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/23255
[SPARK-26307] [SQL] Fix CTAS when INSERT a partitioned table using Hive
serde
## What changes were proposed in this pull request?
This was a Spark 2.3 regression introduced in
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/23254
Unrelated, filed jira: https://issues.apache.org/jira/browse/SPARK-26306
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23242
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22683
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23242
**[Test build #99831 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99831/testReport)**
for PR 23242 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23242
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r239846601
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -37,6 +37,8 @@ displayTitle: Spark SQL Upgrading Guide
- In Spark version 2.4 and earlier,
Github user misutoth commented on the issue:
https://github.com/apache/spark/pull/22331
So I am considering this as the recommended way to read a file sink's
output. If there is a need to include the protocol in this PR as an alternative
we can still reopen it.
---
Github user misutoth closed the pull request at:
https://github.com/apache/spark/pull/22331
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23253
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/23225#discussion_r239860885
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleExternalSorter.java ---
@@ -161,6 +161,10 @@ private void writeSortedFile(boolean
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17654
Gentle ping here, it's out of sync with master if you've got the time to
bring it up to date that would be great.
---
-
To
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22514#discussion_r239887300
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateHiveTableAsSelectCommand.scala
---
@@ -95,9 +77,116 @@ case class
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/23208
@cloud-fan, what are you suggesting to use as a design? If you think this
shouldn't mirror the read side, then let's be clear on what it should look
like. Maybe that's a design doc, or maybe that's
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r239887472
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -130,21 +159,60 @@ class StringIndexer @Since("1.4.0") (
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r239889869
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -310,11 +439,23 @@ object StringIndexerModel extends
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r239885097
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -79,26 +81,53 @@ private[feature] trait StringIndexerBase extends
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r239888373
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -130,21 +159,60 @@ class StringIndexer @Since("1.4.0") (
1 - 100 of 428 matches
Mail list logo