Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13909
**[Test build #69728 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69728/consoleFull)**
for PR 13909 at commit
Github user a-roberts commented on the issue:
https://github.com/apache/spark/pull/15736
New data for us, inlined comparator scores here (code provided below to
check I've not profiled something useless!):
```
ScalaSparkPagerank 2016-12-05 13:44:41 25992811548.149
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13909
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69728/
Test PASSed.
---
GitHub user peterableda opened a pull request:
https://github.com/apache/spark/pull/16172
[SPARK-18740] Log spark.app.name in driver logs
## What changes were proposed in this pull request?
Added simple logInfo line to print out the `spark.app.name` in the driver
logs
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69730/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16168
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/16172
**[Test build #69731 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69731/consoleFull)**
for PR 16172 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13909
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/16169
cc @felixcheung @jkbradley
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15722
@jiexiong PR descriptions are used in git commit messages, and should be
clear and concise. The fix LGTM, but the description should be improved for
future reference. How about we change it into
Github user nsyca commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91104224
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -221,11 +223,14 @@ case class CatalogTable(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16168
**[Test build #69730 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69730/consoleFull)**
for PR 16168 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13909
@cloud-fan thank you for your thoughtful review comments. The latest one
looks better.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user nsyca commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91102867
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -557,9 +557,12 @@ class SessionCatalog(
* If
Github user omuravskiy commented on the issue:
https://github.com/apache/spark/pull/16006
I added new commit, the default value of `backpressureInitialRate` if
`spark.streaming.backpressure.initialRate` is now taken from
`spark.streaming.backpressure.pid.minRate`, or 100 if that is
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/16175
@gatorsmile Could you please take a look when you have time? Thanks a lot!!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16161
**[Test build #69741 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69741/consoleFull)**
for PR 16161 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16174#discussion_r91144141
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -2350,6 +2350,16 @@ object SparkContext extends Logging {
}
}
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16174#discussion_r91141519
--- Diff:
streaming/src/test/java/org/apache/spark/streaming/LocalJavaStreamingContext.java
---
@@ -28,6 +29,7 @@
@Before
public
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16173
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/16173
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69732/
Test FAILed.
---
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16006#discussion_r91118893
--- Diff:
external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/DirectKafkaInputDStream.scala
---
@@ -143,9 +147,14 @@ private[spark]
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16092
**[Test build #69735 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69735/consoleFull)**
for PR 16092 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91131668
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -557,9 +557,12 @@ class SessionCatalog(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91131928
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -221,11 +223,14 @@ case class CatalogTable(
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r91134199
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1047,6 +1047,53 @@ case class
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16014#discussion_r91135457
--- Diff: dev/create-release/release-build.sh ---
@@ -221,14 +235,13 @@ if [[ "$1" == "package" ]]; then
# We increment the Zinc port each time
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16168
also cc @cloud-fan
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16174
> This is resource intensive and it can lead to unneeded test failures
(flakyness) when park.driver.allowMultipleContexts is disabled (this happens
when the order of tests changes).
Do you
GitHub user huaxingao opened a pull request:
https://github.com/apache/spark/pull/16175
[SPARK-17460][SQL]check if statistics.sizeInBytes >=0 in canBroadcast
## What changes were proposed in this pull request?
1. In SparkStrategies.canBroadcast, I will add the check
Github user zhzhan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16068#discussion_r91142141
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveUDFSuite.scala
---
@@ -487,6 +488,52 @@ class HiveUDFSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16175
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16173
**[Test build #69734 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69734/consoleFull)**
for PR 16173 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16138
**[Test build #69738 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69738/consoleFull)**
for PR 16138 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91133644
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -509,32 +509,42 @@ class Analyzer(
* Replaces
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91133855
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -557,9 +557,12 @@ class SessionCatalog(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16128
**[Test build #69740 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69740/consoleFull)**
for PR 16128 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16154#discussion_r91141403
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -167,7 +167,7 @@ private[sql] object SQLUtils extends Logging {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91133302
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -509,32 +509,42 @@ class Analyzer(
* Replaces
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r91134077
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1047,6 +1047,53 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16138
**[Test build #69739 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69739/consoleFull)**
for PR 16138 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16168
@jiangxb1987 We need a low level design about the changes you made here,
especially document the issue you hit and the solution you choose.
---
If your project is set up for it, you can reply
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16172
LGTM. Merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16172
---
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/16128
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 davies commented on the issue:
https://github.com/apache/spark/pull/15923
ping @JoshRosen
---
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 shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16154#discussion_r91136911
--- Diff: core/src/main/scala/org/apache/spark/api/r/RRunner.scala ---
@@ -152,7 +152,7 @@ private[spark] class RRunner[U](
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16154#discussion_r91018557
--- Diff: core/src/main/scala/org/apache/spark/api/r/JVMObjectTracker.scala
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16154#discussion_r91145714
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -247,7 +247,7 @@ private[sql] object SQLUtils extends Logging {
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16154#discussion_r91136204
--- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala
---
@@ -143,12 +142,8 @@ private[r] class RBackendHandler(server: RBackend)
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16154#discussion_r91140911
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -158,7 +158,7 @@ private[sql] object SQLUtils extends Logging {
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/16166#discussion_r91147605
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/ProgressReporter.scala
---
@@ -78,6 +79,8 @@ trait ProgressReporter extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16173
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69734/
Test PASSed.
---
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/9
ping?
---
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 sethah commented on the issue:
https://github.com/apache/spark/pull/15628
ping @dbtsai :)
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/16187
cc @cloud-fan
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/16183
Merging this to master and 2.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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16182
**[Test build #69771 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69771/consoleFull)**
for PR 16182 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16183
---
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 tdas commented on the issue:
https://github.com/apache/spark/pull/16182
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, or
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91234733
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -432,6 +435,57 @@ private[spark] class Executor(
}
/**
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16171
**[Test build #69777 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69777/consoleFull)**
for PR 16171 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91234634
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -229,9 +230,11 @@ private[spark] class Executor(
//
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16121#discussion_r91234619
--- Diff: python/pyspark/serializers.py ---
@@ -278,50 +278,51 @@ def __repr__(self):
return "AutoBatchedSerializer(%s)" % self.serializer
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16121#discussion_r91234665
--- Diff: python/pyspark/serializers.py ---
@@ -278,50 +278,51 @@ def __repr__(self):
return "AutoBatchedSerializer(%s)" % self.serializer
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16121#discussion_r91232276
--- Diff: python/pyspark/serializers.py ---
@@ -96,7 +96,7 @@ def load_stream(self, stream):
raise NotImplementedError
def
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91234685
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -192,13 +189,17 @@ private[spark] class Executor(
serializedTask:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16189
**[Test build #69776 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69776/consoleFull)**
for PR 16189 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91235005
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -161,12 +163,7 @@ private[spark] class Executor(
* @param
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91235062
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -432,6 +435,57 @@ private[spark] class Executor(
}
/**
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91235259
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
---
@@ -448,19 +476,105 @@ class SQLViewSuite extends
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/16173
ok, the BroadcastFactory' comment shows `SparkContext uses a user-specified
BroadcastFactory implementation to instantiate a particular broadcast for
the
entire Spark job.` so I think it
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/16168
@hvanhovell @nsyca @gatorsmile @rxin Thank you for your suggestions! I will
try to make a better approach ASAP! Thank youï¼
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16187
**[Test build #69772 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69772/consoleFull)**
for PR 16187 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16187
**[Test build #69779 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69779/consoleFull)**
for PR 16187 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15422
@zsxwing shouldn't we at least log the exception?
---
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 zhuangxue opened a pull request:
https://github.com/apache/spark/pull/16191
spark decision tree
What algorithm is used in spark decision tree (is ID3, C4.5 or CART)?
You can merge this pull request into a Git repository by running:
$ git pull
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16192
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 foxish commented on the issue:
https://github.com/apache/spark/pull/16092
Thanks @vanzin :)
---
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/16159
Looks like this one breaks Scala 2.10 compile jobs:
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Compile/job/spark-master-compile-sbt-scala-2.10/3301/
---
If your project is set up for
GitHub user anabranch opened a pull request:
https://github.com/apache/spark/pull/16180
[DOCS][MINOR] Clarify Where AccumulatorV2s are Displayed
## What changes were proposed in this pull request?
This PR clarifies where accumulators will be displayed.
## How was
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16159
I don't know how to fix it...
---
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 merlintang commented on the issue:
https://github.com/apache/spark/pull/15819
do you exit the spark shell ? I have tested on this, and this staging file
would be removed after we exit the spark shell under spark 2.0.x.
meanwhile, the staging file are used for
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16178
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 tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/16109#discussion_r91210556
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaTestUtils.scala
---
@@ -286,36 +286,56 @@ class KafkaTestUtils extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16183
**[Test build #69764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69764/consoleFull)**
for PR 16183 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/16173
It existed to support several _internal_ implementations, of which there is
only one now. I don't know that it was ever intended for external
implementations.
---
If your project is set up for it,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16178
**[Test build #69751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69751/consoleFull)**
for PR 16178 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16171
**[Test build #69768 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69768/consoleFull)**
for PR 16171 at commit
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/16182
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, or
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16180
**[Test build #69770 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69770/consoleFull)**
for PR 16180 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16180
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/16180
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69770/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16165
**[Test build #69773 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69773/consoleFull)**
for PR 16165 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/16131
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 jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91234885
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91234781
--- Diff: core/src/test/scala/org/apache/spark/JobCancellationSuite.scala
---
@@ -209,6 +209,41 @@ class JobCancellationSuite extends SparkFunSuite with
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91234770
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91235152
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -126,6 +146,55 @@ private[hive] class
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r91235127
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -432,6 +435,57 @@ private[spark] class Executor(
}
/**
1 - 100 of 596 matches
Mail list logo