Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17633#discussion_r118207319
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -589,18 +590,39 @@ private[client] class Shim_v0_13 extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18085
**[Test build #77298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77298/testReport)**
for PR 18085 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18085
**[Test build #77299 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77299/testReport)**
for PR 18085 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18085
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77299/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18085
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/18068
LGTM, merged into master and branch-2.2. Thanks for all.
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17633
it's really hard to review the tests... Can we just add some simple tests
and refactor the test suites in a follow-up PR?
---
If your project is set up for it, you can reply to this email and
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18009#discussion_r118209944
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DatasetPrimitiveSuite.scala ---
@@ -28,6 +28,8 @@ case class SeqClass(s: Seq[Int])
case
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17308
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 a diff in the pull request:
https://github.com/apache/spark/pull/17891#discussion_r118221814
--- Diff: python/pyspark/ml/tests.py ---
@@ -807,6 +807,18 @@ def test_logistic_regression(self):
except OSError:
pass
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18085
LGTM, merged into master and branch-2.2. 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17308
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77297/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18085
**[Test build #77299 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77299/testReport)**
for PR 18085 at commit
Github user eatoncys commented on the issue:
https://github.com/apache/spark/pull/18060
@SparkQA Retest this please,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 yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18068#discussion_r118231105
--- Diff: python/pyspark/ml/tests.py ---
@@ -1075,7 +1076,8 @@ def test_linear_regression_summary(self):
pValues = s.pValues
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18085
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17308
**[Test build #77297 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77297/testReport)**
for PR 17308 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18014
I took a look at `ColumnVector.getArray`, seems it's already no cost? The
writing needs some copy though.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18085
**[Test build #77298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77298/testReport)**
for PR 18085 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18085
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77298/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18085
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 rxin opened a pull request:
https://github.com/apache/spark/pull/18087
[SPARK-20867][SQL] Move hints from Statistics into HintInfo class
## What changes were proposed in this pull request?
This is a follow-up to SPARK-20857 to move the broadcast hint from
Statistics
GitHub user bogdanrdc opened a pull request:
https://github.com/apache/spark/pull/18086
[SPARK-20854][SQL] Extend hint syntax to support expressions
## What changes were proposed in this pull request?
SQL hint syntax:
* support expressions such as strings, numbers, etc.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18068
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18009#discussion_r118210207
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DatasetPrimitiveSuite.scala ---
@@ -28,6 +28,8 @@ case class SeqClass(s: Seq[Int])
case
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/18082
Hm I'm not sure if it is a good idea to run so many "unit test" style tests
for expressions in the end to end suites. It takes a lot of time than just
running unit tests.
---
If your project is set
GitHub user zero323 opened a pull request:
https://github.com/apache/spark/pull/18085
[SPARK-20631][FOLLOW-UP] Fix incorrect tests.
## What changes were proposed in this pull request?
- Fix incorrect tests for `_check_thresholds`.
- Move test to `ParamTests`.
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17891
@jkbradley It shouldn't. It is not a correct test #18085
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18073
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77296/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18073
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/18073
**[Test build #77296 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77296/testReport)**
for PR 18073 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18087
**[Test build #77301 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77301/testReport)**
for PR 18087 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18086
**[Test build #77300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77300/testReport)**
for PR 18086 at commit
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118263506
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -792,7 +887,18 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18086
**[Test build #77300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77300/testReport)**
for PR 18086 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18081
LGTM, merged into master/branch-2.2/branch-2.1. Thanks for all.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18081
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16989
**[Test build #77302 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77302/testReport)**
for PR 16989 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18058#discussion_r118283005
--- Diff: python/pyspark/ml/fpm.py ---
@@ -49,6 +49,32 @@ def getMinSupport(self):
return self.getOrDefault(self.minSupport)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18090
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18086
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77300/
Test PASSed.
---
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12414
@felixcheung @actuaryzhang Would you mind to have a look at this? 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
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18073
@cloud-fan My dev environment is not convenient to run GUI-based tools like
jconsole. I use a command-line tool
[jvmtop](https://github.com/patric-r/jvmtop).
Screen shots (the column "#T"
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/17113
@jerryshao sorry my delay on this, we have rough design what we want to do
for future changes but I think those are going to take a while and in the mean
time I think this is a useful addition
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/18089
[SPARK-19281][FOLLOWUP][ML] Minor fix for PySpark FPGrowth.
## What changes were proposed in this pull request?
Minor fix for PySpark ```FPGrowth```.
## How was this patch tested?
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/17113#discussion_r118261422
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -145,6 +146,75 @@ private[scheduler] class BlacklistTracker (
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/17113#discussion_r118260480
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -145,6 +146,75 @@ private[scheduler] class BlacklistTracker (
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17967
Personally, I would prefer a HTML list or table one. But I am fine with the
current status if this is okay to all of you here (as I guess none of them is
particularly better given all the
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/17113#discussion_r118260789
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -145,6 +146,75 @@ private[scheduler] class BlacklistTracker (
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118272605
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -163,6 +170,11 @@ final class
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118272761
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -175,33 +187,49 @@ final class
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118274669
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -629,7 +736,9 @@ class CodegenContext
GitHub user ConeyLiu opened a pull request:
https://github.com/apache/spark/pull/18090
[SPARK-20250][Core]Improper OOM error when a task been killed while
spilling data
## What changes were proposed in this pull request?
Currently, when a task is calling spill() but
Github user kishorvpatil commented on the issue:
https://github.com/apache/spark/pull/15009
@vanzin I added `waitFor` method to ChildSparkHandle allowing it to wait
for launched process or thread. Now the test waits for thread to finish.
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18086
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/17967
@actuaryzhang Thanks for your clarification, it makes sense. This looks
good to me. @HyukjinKwon @felixcheung What do you think of the documentation
issue?
---
If your project is set up for
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17770#discussion_r118261213
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -174,17 +174,19 @@ class Dataset[T] private[sql](
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18058#discussion_r118264465
--- Diff: python/pyspark/ml/fpm.py ---
@@ -49,6 +49,32 @@ def getMinSupport(self):
return self.getOrDefault(self.minSupport)
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118272653
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -175,33 +187,49 @@ final class
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118272532
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -287,4 +287,10 @@ package object config {
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118274597
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -792,7 +887,18 @@ class
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18089#discussion_r118280610
--- Diff: python/pyspark/ml/fpm.py ---
@@ -23,17 +23,17 @@
__all__ = ["FPGrowth", "FPGrowthModel"]
-class HasSupport(Params):
GitHub user wselwood opened a pull request:
https://github.com/apache/spark/pull/18088
[Minor] document edge case of updateFunc usage
## What changes were proposed in this pull request?
Include documentation of the fact that the updateFunc is sometimes called
with no new
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18088
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 bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118274094
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -233,10 +223,129 @@ class
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118274179
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -233,10 +223,129 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18089
**[Test build #77303 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77303/testReport)**
for PR 18089 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18089
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77303/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18089
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 kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118300406
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -233,10 +223,124 @@ class CodegenContext
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118302075
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -299,6 +297,9 @@ case class SampleExec(
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118302508
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -233,10 +223,124 @@ class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118305148
--- Diff: docs/configuration.md ---
@@ -520,6 +520,14 @@ Apart from these, the following properties are also
available, and may be useful
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118305105
--- Diff:
core/src/test/scala/org/apache/spark/storage/ShuffleBlockFetcherIteratorSuite.scala
---
@@ -401,4 +411,61 @@ class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18079#discussion_r118305063
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/TableIdentifierParserSuite.scala
---
@@ -49,7 +49,7 @@ class
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/12414#discussion_r118307768
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/RFormula.scala
---
@@ -163,12 +163,20 @@ class RFormula @Since("1.5.0") (@Since("1.5.0")
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16989
**[Test build #77302 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77302/testReport)**
for PR 16989 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18079
Could you check the alias precedence of the other database?
```SQL
select col1 as a, col2 as b from t1 as (c, d);
```
Which alias should be used as the output schema? `(a, b)`
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/18075#discussion_r118320606
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -233,10 +223,124 @@ class CodegenContext
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18087
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/18087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77301/
Test PASSed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18091
cc @joshRosan
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18073
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/17967
I think a html table is better?
https://github.com/apache/spark/pull/17967#discussion_r117917444
+ @srowen for your opinion- to be honest I don't think I've actually seen a
table in Spark
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/18051
Improvements to R API doc online can be useful.
But I think that is somewhat independent of this change/PR. Let's focus on
your points on `Cleaning duplicate links` and `Trying to clean see
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/18051
If we consider improvement of the online documentation to be a separate
problem, then I fully agree with @actuaryzhang.
---
If your project is set up for it, you can reply to this email and have
Github user kevinyu98 commented on the issue:
https://github.com/apache/spark/pull/12646
test it please. 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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/12414#discussion_r118306841
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/RFormula.scala
---
@@ -163,12 +163,20 @@ class RFormula @Since("1.5.0") (@Since("1.5.0")
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16989
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/16989
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77302/
Test FAILed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18067#discussion_r118315143
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -776,6 +778,19 @@ newDF <- createDataFrame(data.frame(x = c(1.5, 3.2)))
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17633
**[Test build #77306 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77306/testReport)**
for PR 17633 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18087
**[Test build #77301 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77301/testReport)**
for PR 18087 at commit
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/18091
[SPARK-20868][CORE] UnsafeShuffleWriter should verify the position after
FileChannel.transferTo
## What changes were proposed in this pull request?
Long time ago we fixed a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16989#discussion_r118304766
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -154,7 +161,7 @@ final class
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/17750#discussion_r118310789
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -158,7 +158,7
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/17113#discussion_r118315661
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -145,6 +146,75 @@ private[scheduler] class BlacklistTracker (
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/17633#discussion_r118318733
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -589,18 +590,34 @@ private[client] class Shim_v0_13 extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18089
**[Test build #77303 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77303/testReport)**
for PR 18089 at commit
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/18089
Thanks @yanboliang
---
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,
1 - 100 of 514 matches
Mail list logo