Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212756422
**[Test build #56485 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56485/consoleFull)**
for PR 12556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212756425
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212756424
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212756240
**[Test build #56485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56485/consoleFull)**
for PR 12556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212754891
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
Github user jodersky commented on the pull request:
https://github.com/apache/spark/pull/12524#issuecomment-212754822
is this related to #12436 ?
---
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 pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212754896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212754203
**[Test build #56475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56475/consoleFull)**
for PR 10024 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12551
---
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 davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12490#discussion_r60529161
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SortPrefixUtils.scala ---
@@ -66,6 +66,32 @@ object SortPrefixUtils {
}
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12490#discussion_r60529190
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/Sort.scala
---
@@ -139,11 +148,15 @@ case class Sort(
val dataSize =
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212753448
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12551#issuecomment-212753365
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
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12490#discussion_r60528873
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SortPrefixUtils.scala ---
@@ -66,6 +66,32 @@ object SortPrefixUtils {
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12551#issuecomment-212752845
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12551#issuecomment-212752841
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
Github user xwu0226 closed the pull request at:
https://github.com/apache/spark/pull/12406
---
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 xwu0226 commented on the pull request:
https://github.com/apache/spark/pull/12406#issuecomment-212752759
Thanks @wangmiao1981 .
This happens again. all the commits after my last commits got pulled into
this PR. I need to close it and open a new PR. Will submit a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12551#issuecomment-212752681
**[Test build #56470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56470/consoleFull)**
for PR 12551 at commit
Github user wangmiao1981 commented on the pull request:
https://github.com/apache/spark/pull/12406#issuecomment-212752658
@xwu0226 use git rebase upstream/master. Do not use git merge
upstream/master. I have the same issue before. git merge will add others'
commits to your PR. git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12560#issuecomment-212752513
**[Test build #56484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56484/consoleFull)**
for PR 12560 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12490#discussion_r60528592
--- Diff:
core/src/test/scala/org/apache/spark/util/collection/unsafe/sort/PrefixComparatorsSuite.scala
---
@@ -110,4 +112,12 @@ class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212752316
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212752210
**[Test build #56478 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56478/consoleFull)**
for PR 12556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212752313
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12559#issuecomment-212751784
**[Test build #56483 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56483/consoleFull)**
for PR 12559 at commit
GitHub user wangmiao1981 opened a pull request:
https://github.com/apache/spark/pull/12560
[SPARK-14571][ML]Log instrumentation in ALS
## What changes were proposed in this pull request?
Add log instrumentation for parameters:
rank, numUserBlocks, numItemBlocks,
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212751532
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12559#issuecomment-212750650
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12559#issuecomment-212750651
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12559#issuecomment-212750649
**[Test build #56481 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56481/consoleFull)**
for PR 12559 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12557#issuecomment-212750487
**[Test build #56482 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56482/consoleFull)**
for PR 12557 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12559#issuecomment-212750481
**[Test build #56481 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56481/consoleFull)**
for PR 12559 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12553#issuecomment-212750214
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12553#issuecomment-212750212
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12553#issuecomment-212750048
**[Test build #56468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56468/consoleFull)**
for PR 12553 at commit
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/12559
[SPARK-14793][SQL] Code generation for large complex type exceeds JVM size
limit.
## What changes were proposed in this pull request?
Code generation for complex type, `CreateArray`,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11800#issuecomment-212749912
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11800#issuecomment-212749911
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11800#issuecomment-212749831
**[Test build #56474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56474/consoleFull)**
for PR 11800 at commit
Github user agsachin commented on the pull request:
https://github.com/apache/spark/pull/12357#issuecomment-212749355
hey closed this thread as we are moving towards approach 2 explained in the
jira https://issues.apache.org/jira/browse/SPARK-14597
---
If your project is set up for
Github user agsachin closed the pull request at:
https://github.com/apache/spark/pull/12357
---
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12494#issuecomment-212748601
@viirya Thanks for bearing with me.
---
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/12494#issuecomment-212748355
@HyukjinKwon yea thanks for comment.
I will remove the null check then. And I should make change to CSVOption to
avoid the null exception.
---
If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12557#issuecomment-212748073
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
Github user lianhuiwang commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212748052
@davies Now all tests have been passed. So Could you take a look again?
Thanks.
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12557#issuecomment-212748076
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12557#issuecomment-212748057
**[Test build #56480 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56480/consoleFull)**
for PR 12557 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212747490
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212747494
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12239#discussion_r60527018
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -414,8 +414,42 @@ class Analyzer(
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212747343
**[Test build #56466 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56466/consoleFull)**
for PR 10024 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12239#discussion_r60526988
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -414,8 +414,42 @@ class Analyzer(
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12557#issuecomment-212747247
**[Test build #56480 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56480/consoleFull)**
for PR 12557 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12494#issuecomment-212747103
@viirya Ah, thanks! however, I am a bit worried if disallowing `null` as a
value in options is correct. I don't want to be kind of so picky but I think it
might
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12490#discussion_r60526773
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/PrefixComparators.java
---
@@ -28,88 +28,84 @@
public class PrefixComparators
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/8374#issuecomment-212746977
Seems no because it gets stale. If nobody takes this, I'll do it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12239#discussion_r60526769
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -414,8 +414,42 @@ class Analyzer(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212746211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212746210
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12525#issuecomment-212746170
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12525#issuecomment-212746169
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212746062
**[Test build #56462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56462/consoleFull)**
for PR 10024 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12525#issuecomment-212746059
**[Test build #56467 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56467/consoleFull)**
for PR 12525 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12490#discussion_r60526550
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleInMemorySorter.java ---
@@ -87,18 +102,18 @@ public void expandPointerArray(LongArray
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212745547
**[Test build #56472 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56472/consoleFull)**
for PR 12556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212745631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212745630
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
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/12494#issuecomment-212745533
@HyukjinKwon ok. make thing clearer...
@mathieulongtin wants to use `null` as option value in spark-csv from
databricks. So he did the PR to allow passing
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12555#issuecomment-212745308
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12555#issuecomment-212745302
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12555#issuecomment-212744664
**[Test build #56463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56463/consoleFull)**
for PR 12555 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60526334
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -238,4 +281,42 @@ class
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/12552#issuecomment-212744081
@HyukjinKwon thanks for picking up this PR & taking the time to investigate
the places where the changes could be useful for Spark. This looks good to me
as well.
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60526194
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -238,4 +281,42 @@ class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/12494#discussion_r60526159
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -367,16 +370,19 @@ def format(self, source):
@since(1.5)
def option(self, key, value):
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60526133
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/BenchmarkWholeStageCodegen.scala
---
@@ -224,6 +224,127 @@ class BenchmarkWholeStageCodegen
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12558#issuecomment-212742755
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12558#issuecomment-212742754
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12558#issuecomment-212742683
**[Test build #56473 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56473/consoleFull)**
for PR 12558 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60526044
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -69,17 +90,29 @@ class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12547#issuecomment-212742481
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
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60525999
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -69,17 +90,29 @@ class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12547#issuecomment-212742482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60525942
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -69,17 +90,29 @@ class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12547#issuecomment-212742410
**[Test build #56479 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56479/consoleFull)**
for PR 12547 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60525850
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -40,12 +41,32 @@ import
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12440#discussion_r60525541
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -271,6 +271,74 @@ class CodegenContext {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212739055
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212739052
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12525#discussion_r60525166
--- Diff: core/src/main/scala/org/apache/spark/scheduler/StageInfo.scala ---
@@ -36,7 +36,7 @@ class StageInfo(
val rddInfos: Seq[RDDInfo],
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10024#issuecomment-212738764
**[Test build #56459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56459/consoleFull)**
for PR 10024 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212737750
**[Test build #56478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56478/consoleFull)**
for PR 12556 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12556#issuecomment-212737840
cc @hvanhovell
---
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 pull request:
https://github.com/apache/spark/pull/12547#issuecomment-212737812
**[Test build #56479 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56479/consoleFull)**
for PR 12547 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12525#discussion_r60524791
--- Diff: core/src/main/scala/org/apache/spark/executor/TaskMetrics.scala
---
@@ -244,7 +243,14 @@ private[spark] class ListenerTaskMetrics(accumUpdates:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12525#issuecomment-212735921
**[Test build #56477 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56477/consoleFull)**
for PR 12525 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12525#issuecomment-212733494
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12550
---
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 dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/12547#discussion_r60524553
--- Diff: R/pkg/R/context.R ---
@@ -225,3 +225,17 @@ broadcast <- function(sc, object) {
setCheckpointDir <- function(sc, dirName) {
1 - 100 of 1201 matches
Mail list logo