Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14008
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14095
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/14008
Thanks - merging in master/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 and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14108
**[Test build #61999 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61999/consoleFull)**
for PR 14108 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14095
Thanks - merging in 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
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14108#discussion_r70144675
--- Diff: dev/create-release/release-build.sh ---
@@ -258,7 +258,7 @@ if [[ "$1" == "publish-snapshot" ]]; then
-Phive-thriftserver deploy
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14108#discussion_r70144693
--- Diff: dev/create-release/release-build.sh ---
@@ -258,7 +258,7 @@ if [[ "$1" == "publish-snapshot" ]]; then
-Phive-thriftserver deploy
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14108#discussion_r70144390
--- Diff: dev/create-release/release-build.sh ---
@@ -258,7 +258,7 @@ if [[ "$1" == "publish-snapshot" ]]; then
-Phive-thriftserver deploy
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14108#discussion_r70144266
--- Diff: dev/create-release/release-build.sh ---
@@ -258,7 +258,7 @@ if [[ "$1" == "publish-snapshot" ]]; then
-Phive-thriftserver deploy
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14108
**[Test build #61998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61998/consoleFull)**
for PR 14108 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14108#discussion_r70144037
--- Diff: dev/create-release/release-build.sh ---
@@ -258,7 +258,7 @@ if [[ "$1" == "publish-snapshot" ]]; then
-Phive-thriftserver deploy
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/14108
cc @JoshRosen @rxin
---
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 yhuai opened a pull request:
https://github.com/apache/spark/pull/14108
[SPARK-16453] [BUILD] release-build.sh is missing hive-thriftserver for
scala 2.10
## What changes were proposed in this pull request?
This PR adds hive-thriftserver profile to scala 2.10 build
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14034
**[Test build #61997 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61997/consoleFull)**
for PR 14034 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14107
**[Test build #61996 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61996/consoleFull)**
for PR 14107 at commit
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/14107
[SPARK-16387][SQL] JDBC Writer should use dialect to quote field names.
## What changes were proposed in this pull request?
Currently, JDBC Writer uses dialects to get datatypes, but
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14008
**[Test build #3173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3173/consoleFull)**
for PR 14008 at commit
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/13890#discussion_r70139445
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExistingRDD.scala ---
@@ -74,13 +74,71 @@ object RDDConversions {
}
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61992/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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/13991
**[Test build #61992 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61992/consoleFull)**
for PR 13991 at commit
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/14006
@mengxr
---
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/13831
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61991/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13831
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/13831
**[Test build #61991 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61991/consoleFull)**
for PR 13831 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13374
**[Test build #3172 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3172/consoleFull)**
for PR 13374 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/13738
Sorry for my delay on this, so it looks like if we create any FileSystem is
will cause the service loaders to load all services, so in hadoopRDD,
newApiHadoopRDD where we don't know the path we
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61995/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61995/consoleFull)**
for PR 14082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14095#discussion_r70130356
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -228,6 +228,15 @@ class Dataset[T] private[sql](
}
}
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14095#discussion_r70130150
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -228,6 +228,15 @@ class Dataset[T] private[sql](
}
}
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14093
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/14093
Merging in master/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 and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61993/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61993 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61993/consoleFull)**
for PR 14082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14006
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/14006
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61994/
Test PASSed.
---
Github user lovexi commented on the issue:
https://github.com/apache/spark/pull/14080
Added some test cases to verify expected metrics values @ericl
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14006
**[Test build #61994 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61994/consoleFull)**
for PR 14006 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14093
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/14093
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61988/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14093
**[Test build #61988 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61988/consoleFull)**
for PR 14093 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61995/consoleFull)**
for PR 14082 at commit
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70125600
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70125056
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70124885
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70124300
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70124201
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70124162
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61993 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61993/consoleFull)**
for PR 14082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14006
**[Test build #61994 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61994/consoleFull)**
for PR 14006 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14075
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/14075
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61990/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14075
**[Test build #61990 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61990/consoleFull)**
for PR 14075 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14048
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/14048
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61989/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14048
**[Test build #61989 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61989/consoleFull)**
for PR 14048 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70122513
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70122361
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13374
---
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 keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70122204
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70122235
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14008
**[Test build #3173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3173/consoleFull)**
for PR 14008 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70121939
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13374
Merging in 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
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70121544
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/14087#discussion_r70121449
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamReader.scala
---
@@ -281,6 +281,31 @@ final class DataStreamReader
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70121398
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,198 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61992 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61992/consoleFull)**
for PR 13991 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13969#discussion_r70120022
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Reflect.scala
---
@@ -0,0 +1,170 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13374
**[Test build #3171 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3171/consoleFull)**
for PR 13374 at commit
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
@hvanhovell can you take a look at this?
---
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 keypointt commented on the issue:
https://github.com/apache/spark/pull/14097
maybe I should also cc @tdas , he is on the streaming part
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13867
cc @hvanhovell @cloud-fan 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
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13969#discussion_r70118085
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Reflect.scala
---
@@ -0,0 +1,170 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14034#discussion_r70118783
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -46,6 +46,21 @@ trait CheckAnalysis extends
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13969
I've pushed a new commit that addresses the review comments.
---
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
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13969#discussion_r70117803
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Reflect.scala
---
@@ -0,0 +1,170 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14106
**[Test build #61986 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61986/consoleFull)**
for PR 14106 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14106
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14008
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61987/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14106
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61986/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14008
**[Test build #61987 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61987/consoleFull)**
for PR 14008 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14008
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 petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13969#discussion_r70116854
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Reflect.scala
---
@@ -0,0 +1,170 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13374
**[Test build #3172 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3172/consoleFull)**
for PR 13374 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13831
**[Test build #61991 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61991/consoleFull)**
for PR 13831 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13831
> it would be nice to test using purge actually does a purge
None of the tests run on actual HDFS cluster, so I doubt that would be
possible without a lot more work...
---
If your project
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13831#discussion_r70115433
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -179,7 +179,8 @@ case class DescribeDatabaseCommand(
case
Github user lovexi commented on a diff in the pull request:
https://github.com/apache/spark/pull/14080#discussion_r70115043
--- Diff:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockHandler.java
---
@@ -143,4 +179,26 @@ private void
Github user lovexi commented on a diff in the pull request:
https://github.com/apache/spark/pull/14080#discussion_r70114491
--- Diff:
core/src/main/scala/org/apache/spark/deploy/ExternalShuffleServiceSource.scala
---
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14078
good to go :+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,
Github user lovexi commented on a diff in the pull request:
https://github.com/apache/spark/pull/14080#discussion_r70112425
--- Diff:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockHandler.java
---
@@ -93,18 +113,34 @@ protected void
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14093
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 ioana-delaney commented on the issue:
https://github.com/apache/spark/pull/13867
Can someone please review the changes? Thank you.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14012
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/14012
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61985/
Test PASSed.
---
Github user lovexi commented on a diff in the pull request:
https://github.com/apache/spark/pull/14080#discussion_r70111868
--- Diff:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockHandler.java
---
@@ -64,6 +75,10 @@ public
101 - 200 of 473 matches
Mail list logo