Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/9403#discussion_r43851021
--- Diff:
extras/kinesis-asl/src/test/scala/org/apache/spark/streaming/kinesis/KinesisTestUtils.scala
---
@@ -90,24 +102,44 @@ private[kinesis] class KinesisTes
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/9403#discussion_r43850884
--- Diff:
extras/kinesis-asl/src/test/scala/org/apache/spark/streaming/kinesis/KinesisStreamSuite.scala
---
@@ -56,6 +56,8 @@ class KinesisStreamSuite extends K
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9461#issuecomment-153621309
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 pr
GitHub user huaxingao opened a pull request:
https://github.com/apache/spark/pull/9461
Spark 11474
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/huaxingao/spark spark_11474
Alternatively you can review and apply these changes
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9460#issuecomment-153621022
**[Test build #45006 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45006/consoleFull)**
for PR 9460 at commit
[`7de664a`](https://gith
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/9385#issuecomment-153620226
@dbtsai Thank you!
Please let me know if you need any extra code change.
---
If your project is set up for it, you can reply to this email and have your
re
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9460#issuecomment-153619862
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9460#issuecomment-153619930
Merged build started.
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/9419#discussion_r43850164
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -232,7 +232,7 @@ class Analyzer(
// substi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153618536
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153618538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153618311
**[Test build #44997 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44997/consoleFull)**
for PR 9383 at commit
[`fc5e052`](https://git
GitHub user nishkamravi2 opened a pull request:
https://github.com/apache/spark/pull/9460
[SPARK-11501][CORE][YARN] Propagate spark.rpc config to executors
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nishkamravi2/spark master
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153618028
**[Test build #45005 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45005/consoleFull)**
for PR 9303 at commit
[`af3e7c7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153617149
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153617188
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9358#issuecomment-153616627
**[Test build #45003 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45003/consoleFull)**
for PR 9358 at commit
[`fbf791e`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9449#issuecomment-153616160
**[Test build #45004 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45004/consoleFull)**
for PR 9449 at commit
[`11c28ee`](https://gith
Github user taishi-oss commented on the pull request:
https://github.com/apache/spark/pull/8935#issuecomment-153615685
Sorry for being late.
I rebased this branch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153615498
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9449#issuecomment-153614948
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/9449#issuecomment-153614918
Merged build triggered.
---
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 h
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9185#issuecomment-153614611
sessions are critical - actually one of the most important features for
spark 1.6.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43849661
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/Encoder.scala
---
@@ -37,3 +37,120 @@ trait Encoder[T] extends Serializable {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9287#issuecomment-153613616
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9287#issuecomment-153613617
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9358#issuecomment-153613475
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9358#issuecomment-153613507
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9287#issuecomment-153613432
**[Test build #44999 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44999/consoleFull)**
for PR 9287 at commit
[`84baafc`](https://git
Github user taishi-oss commented on the pull request:
https://github.com/apache/spark/pull/8534#issuecomment-153612968
Sorry for being late.
I rebased this branch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153611121
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9452#issuecomment-153611081
**[Test build #1976 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1976/consoleFull)**
for PR 9452 at commit
[`8ce9696`](https://git
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9445#issuecomment-153611065
cc @marmbrus
---
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
ena
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-15368
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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43848905
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/Encoder.scala
---
@@ -37,3 +35,39 @@ trait Encoder[T] extends Serializable {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153610966
**[Test build #44998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44998/consoleFull)**
for PR 9448 at commit
[`b2a6838`](https://git
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/9403#issuecomment-153606779
test this again.
---
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9390#discussion_r43847164
--- Diff: core/src/main/scala/org/apache/spark/api/r/RUtils.scala ---
@@ -51,15 +59,30 @@ private[spark] object RUtils {
val isYarnClient = master
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9390#discussion_r43847118
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -362,22 +363,41 @@ object SparkSubmit {
}
}
-
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9459#issuecomment-153596557
**[Test build #45002 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45002/consoleFull)**
for PR 9459 at commit
[`91ade3f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153596433
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153596430
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153595974
**[Test build #45001 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45001/consoleFull)**
for PR 9383 at commit
[`1c0c6c3`](https://gith
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/9452#issuecomment-153595837
The last failed test is not related (flaky).
---
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 proje
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/8984#discussion_r43846836
--- Diff: R/pkg/R/types.R ---
@@ -0,0 +1,43 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreeme
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153595672
@JoshRosen @yhuai pushed a refactor on this (reduce possibility of full GC
by re-use the array in map), please take another look.
---
If your project is set up for it, y
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9383#discussion_r43846718
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/AggregationQuerySuite.scala
---
@@ -589,6 +588,13 @@ abstract class AggregationQuerySuit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9459#issuecomment-153594512
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9459#issuecomment-153594565
Merged build started.
---
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 HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/9327#discussion_r43846677
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
---
@@ -314,4 +314,24 @@ class ParquetFilte
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/8840#issuecomment-153592786
@rxin @yhuai OK. Submitted a following pr at #9459.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9383#discussion_r43846549
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorter.java
---
@@ -386,24 +388,37 @@ public void insertKVRecord(Objec
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/9459
[SPARK-10304][SQL] Following up checking valid dir structure for partition
discovery
This patch follows up #8840.
You can merge this pull request into a Git repository by running:
$ git pull ht
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153591839
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153591862
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153590796
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153590823
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9349#issuecomment-153589152
**[Test build #1975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1975/consoleFull)**
for PR 9349 at commit
[`d15e14a`](https://gi
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9443#discussion_r43845783
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2045,3 +2045,34 @@ setMethod("attach",
}
attach(newEnv, pos = pos, name = name, war
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9443#discussion_r43845708
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2045,3 +2045,34 @@ setMethod("attach",
}
attach(newEnv, pos = pos, name = name, war
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9287#issuecomment-153585136
**[Test build #44999 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44999/consoleFull)**
for PR 9287 at commit
[`84baafc`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9287#issuecomment-153584875
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9287#issuecomment-153584886
Merged build started.
---
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 jaceklaskowski commented on the pull request:
https://github.com/apache/spark/pull/9444#issuecomment-153579178
Thanks @rxin! I knew it might've been too small for a change, but since
it's in the UI, I thought I'd *not* wait till I find other typos.
---
If your project is
Github user choochootrain commented on a diff in the pull request:
https://github.com/apache/spark/pull/9367#discussion_r43844511
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -366,6 +367,72 @@ class SparkSubmitSuite
}
}
Github user choochootrain commented on a diff in the pull request:
https://github.com/apache/spark/pull/9367#discussion_r43844348
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -366,6 +367,72 @@ class SparkSubmitSuite
}
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153575588
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153575587
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9303#issuecomment-153575454
**[Test build #44995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44995/consoleFull)**
for PR 9303 at commit
[`af3e7c7`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153574837
**[Test build #44998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44998/consoleFull)**
for PR 9448 at commit
[`b2a6838`](https://gith
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8984#discussion_r43843563
--- Diff: R/pkg/R/types.R ---
@@ -0,0 +1,43 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153573827
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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153573858
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153573843
Merged build triggered.
---
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 h
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43843372
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/Encoder.scala
---
@@ -37,3 +35,39 @@ trait Encoder[T] extends Serializable {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153573130
**[Test build #44997 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44997/consoleFull)**
for PR 9383 at commit
[`fc5e052`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153572263
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/8669#issuecomment-153572262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8669#issuecomment-153572261
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9383#issuecomment-153572253
Merged build triggered.
---
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8669#issuecomment-153572150
**[Test build #44988 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44988/consoleFull)**
for PR 8669 at commit
[`16ea884`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9367#issuecomment-153572088
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9367#issuecomment-153572089
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9367#issuecomment-153572051
[Test build #44991 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44991/console)
for PR 9367 at commit
[`53f7c4c`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153571208
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153571206
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
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43842039
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -441,6 +537,17 @@ class Dataset[T] private(
/** Collects the elements to an
Github user vundela commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153568315
Thanks for the review @vanzin, made the changes as you suggested.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9410
---
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 enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153567900
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9448#issuecomment-153567906
Merged build started.
---
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/9410#issuecomment-153567691
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
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9383#discussion_r43841671
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala
---
@@ -502,44 +511,37 @@ class TungstenAggregati
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43841577
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -441,6 +537,17 @@ class Dataset[T] private(
/** Collects the elements to a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9349#issuecomment-153567183
**[Test build #1975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1975/consoleFull)**
for PR 9349 at commit
[`d15e14a`](https://git
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/9349#issuecomment-153567087
The changes LGTM, could you update the description to reflect the changes?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43841129
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -441,6 +537,17 @@ class Dataset[T] private(
/** Collects the elements to a
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9358#discussion_r43841085
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -441,6 +537,17 @@ class Dataset[T] private(
/** Collects the elements to an
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9367#discussion_r43840818
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -366,6 +367,72 @@ class SparkSubmitSuite
}
}
+ //
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9367#discussion_r43840823
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -366,6 +367,72 @@ class SparkSubmitSuite
}
}
+ //
1 - 100 of 1026 matches
Mail list logo
