Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16323
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70295/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16324
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70296/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16324
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/16323
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16323
**[Test build #70295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70295/consoleFull)**
for PR 16323 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16324
**[Test build #70296 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70296/testReport)**
for PR 16324 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16324
Please add the description in the JIRA to the PR description. FYI, you
still can edit the description after you create the PR.
---
If your project is set up for it, you can reply to this email
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16290
I finished my review. 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
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16301#discussion_r92916892
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -496,9 +508,114 @@ count(carsDF_test)
head(carsDF_test)
```
-
### Models
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/16194
@sethah No. Just for the convenience in repl. It's somewhat confusing when
the model's desc is just the uid of its trainer when I work in repl.
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16249
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70294/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16249
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16249
**[Test build #70294 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70294/testReport)**
for PR 16249 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16290#discussion_r92916658
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -819,7 +819,13 @@ private[sql] class SQLConf extends Serializable
Github user kubatyszko commented on a diff in the pull request:
https://github.com/apache/spark/pull/16319#discussion_r92916668
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -246,9 +246,9 @@ private[csv] object
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70299 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70299/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70298/testReport)**
for PR 16291 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
Jenkins test 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16282
**[Test build #70297 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70297/testReport)**
for PR 16282 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16282
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/16324
**[Test build #70296 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70296/testReport)**
for PR 16324 at commit
GitHub user shenh062326 opened a pull request:
https://github.com/apache/spark/pull/16324
Resolve faile to use UDF that jar file in hdfs.
## What changes were proposed in this pull request?
In SparkContext, setURLStreamHandlerFactory method on URL with an instance
of
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15544
@wzhfy let's close this one for now. I think we can revisit it in the
future when we introduce more advanced statistics.
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70293/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15666
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/15666
**[Test build #70293 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70293/testReport)**
for PR 15666 at commit
Github user bdwyer2 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16290#discussion_r92916155
--- Diff: R/pkg/inst/tests/testthat/test_context.R ---
@@ -72,6 +72,20 @@ test_that("repeatedly starting and stopping
SparkSession", {
}
})
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16290
@felixcheung @bdwyer2 Could you take another look ?
---
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r92916070
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -30,21 +30,44 @@ import
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/16228
To solve [the two
issues](https://github.com/apache/spark/pull/16228#issuecomment-265946299) I
mentioned above, a separate PR is sent
[here](https://github.com/apache/spark/pull/16323).
We need
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16323
**[Test build #70295 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70295/consoleFull)**
for PR 16323 at commit
GitHub user wzhfy opened a pull request:
https://github.com/apache/spark/pull/16323
[SPARK-18911] [SQL] Define CatalogStatistics to interact with metastore and
convert it to Statistics based on cbo switch
## What changes were proposed in this pull request?
Statistics
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16305
---
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 shivaram commented on the issue:
https://github.com/apache/spark/pull/16305
Thanks @HyukjinKwon - Merging into 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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r92915894
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
---
@@ -120,7 +128,10 @@ object Cast {
> SELECT
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r92915851
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -111,7 +112,8 @@ case class CatalogTablePartition(
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16308
I'd rename all the zoneId to timeZoneId to reduce confusion ..
---
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r92915771
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
---
@@ -120,7 +128,10 @@ object Cast {
> SELECT
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r92915757
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
---
@@ -110,6 +110,14 @@ object Cast {
case (_:
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r92915708
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -30,21 +30,44 @@ import
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92915680
--- Diff: core/src/test/scala/org/apache/spark/JobCancellationSuite.scala
---
@@ -209,6 +209,41 @@ class JobCancellationSuite extends SparkFunSuite with
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16030
okay! I'll make a JIRA later and 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 maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16319#discussion_r92915632
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -246,9 +246,9 @@ private[csv] object
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16319
@HyukjinKwon Thanks for your pinging! I left some 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 does not
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16249
**[Test build #70294 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70294/testReport)**
for PR 16249 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/13909#discussion_r92915607
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -56,33 +58,93 @@ case class
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16030
This was the behavior change your PR proposed before, I think it makes
sense, you can send a PR to fix it in Spark 2.2
---
If your project is set up for it, you can reply to this email and have
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16312
I like how they are grouped. not sure why tests are failing though
---
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
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15915
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16305
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70291/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16305
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/16305
**[Test build #70291 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70291/testReport)**
for PR 16305 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16320
Actually, I made the end-to-end test based on the example of the use case
in JIRA first. And, I removed that from here because the current test case is
the minimal version of that.
---
If
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/16320#discussion_r92915371
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -85,7 +85,9 @@ private[csv] object
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16261#discussion_r92914999
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1009,13 +1009,14 @@ class DAGScheduler(
}
val
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914921
--- Diff: core/src/test/scala/org/apache/spark/JobCancellationSuite.scala
---
@@ -209,6 +209,41 @@ class JobCancellationSuite extends SparkFunSuite with
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914965
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -148,9 +161,23 @@ private[spark] class Executor(
}
def
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914792
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -229,9 +259,12 @@ private[spark] class Executor(
//
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914957
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -432,6 +465,93 @@ private[spark] class Executor(
}
/**
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914715
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -161,12 +163,7 @@ private[spark] class Executor(
* @param
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914906
--- Diff: core/src/test/scala/org/apache/spark/JobCancellationSuite.scala
---
@@ -209,6 +209,83 @@ class JobCancellationSuite extends SparkFunSuite with
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914831
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -432,6 +465,93 @@ private[spark] class Executor(
}
/**
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914652
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -148,9 +161,23 @@ private[spark] class Executor(
}
def
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16189#discussion_r92914887
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -432,6 +465,93 @@ private[spark] class Executor(
}
/**
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16263
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70292/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16263
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/16263
**[Test build #70292 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70292/testReport)**
for PR 16263 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16320
@dongjoon-hyun would we need a end-to-end test too?
---
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/16304
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70284/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16304
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16304
**[Test build #70284 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70284/testReport)**
for PR 16304 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16320#discussion_r92914568
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -85,7 +85,9 @@ private[csv] object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16305
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70286/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16305
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 mariusvniekerk commented on the issue:
https://github.com/apache/spark/pull/15666
Rebased.
---
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/15666
**[Test build #70293 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70293/testReport)**
for PR 15666 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16305
**[Test build #70286 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70286/testReport)**
for PR 16305 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16319
Let's put `[WIP]` in the PR title, for example, (`[WIP][SPARK-18699] ... `)
if you are working on this to follow the suggestion in the JIRA. It seems we
need to update the PR description (and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16322
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/16322
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70288/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16322
**[Test build #70288 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70288/testReport)**
for PR 16322 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16263
**[Test build #70292 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70292/testReport)**
for PR 16263 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70290/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16304
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/16291
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/16304
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70282/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70290/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16304
**[Test build #70282 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70282/testReport)**
for PR 16304 at commit
Github user kubatyszko commented on a diff in the pull request:
https://github.com/apache/spark/pull/16319#discussion_r92913976
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -246,9 +246,9 @@ private[csv] object
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16320#discussion_r92913834
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -85,7 +85,9 @@ private[csv] object
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16319#discussion_r92913320
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -246,9 +246,9 @@ private[csv] object
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16319
Can you describe which case this PR fixes? I think you meant
https://issues.apache.org/jira/browse/SPARK-18699
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16305
**[Test build #70291 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70291/testReport)**
for PR 16305 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16305#discussion_r92912445
--- Diff: core/src/main/scala/org/apache/spark/deploy/RPackageUtils.scala
---
@@ -231,8 +236,10 @@ private[deploy] object RPackageUtils extends Logging
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16263#discussion_r92912335
--- Diff: python/pyspark/rdd.py ---
@@ -2349,7 +2352,12 @@ def toLocalIterator(self):
"""
with SCCallSiteSync(self.context) as css:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16305#discussion_r92912156
--- Diff: core/src/main/scala/org/apache/spark/deploy/RPackageUtils.scala
---
@@ -231,8 +236,10 @@ private[deploy] object RPackageUtils extends Logging
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16282
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/16282
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70281/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16282
**[Test build #70281 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70281/testReport)**
for PR 16282 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16305#discussion_r92911599
--- Diff: core/src/main/scala/org/apache/spark/deploy/RPackageUtils.scala
---
@@ -231,8 +236,10 @@ private[deploy] object RPackageUtils extends Logging {
1 - 100 of 470 matches
Mail list logo
