Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44357472
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Count.scala
---
@@ -32,23 +32,34 @@ case class Count(child: Expressio
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9559#discussion_r44357467
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -193,6 +258,16 @@ private[spark] object HighlyCompressedMapStatus {
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9092
---
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 andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9559#discussion_r44357262
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -141,7 +202,7 @@ private[spark] class HighlyCompressedMapStatus private (
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9559#discussion_r44357246
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -121,6 +123,65 @@ private[spark] class CompressedMapStatus(
/**
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9559#discussion_r44357257
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -121,6 +123,65 @@ private[spark] class CompressedMapStatus(
/**
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9559#discussion_r44357152
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -121,6 +123,65 @@ private[spark] class CompressedMapStatus(
/**
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248527
**[Test build #45451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45451/consoleFull)**
for PR 9579 at commit
[`635bbe7`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9428#issuecomment-155248415
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/9579#issuecomment-155248535
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9092#issuecomment-155248366
merging with master, branch-1.6
Thank you for the PR!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9428#issuecomment-155248316
**[Test build #45431 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45431/consoleFull)**
for PR 9428 at commit
[`676317b`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248221
**[Test build #45451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45451/consoleFull)**
for PR 9579 at commit
[`635bbe7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248162
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/9513#issuecomment-155248180
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/9579#issuecomment-155248181
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/9513#issuecomment-155248167
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/9559#issuecomment-155247985
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9513#issuecomment-155248084
@feynmanliang I think that's the last fix.
Thinking more about it, I'm on board with changing LDAModel to be abstract,
as long as it's a minor change. I'll se
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9559#issuecomment-155247982
**[Test build #45450 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45450/consoleFull)**
for PR 9559 at commit
[`e1d1106`](https://git
Github user olarayej commented on the pull request:
https://github.com/apache/spark/pull/8984#issuecomment-155247867
I have created a new branch and PR #9579 to follow up on this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user olarayej opened a pull request:
https://github.com/apache/spark/pull/9579
[SPARK-10863][SPARKR] Method coltypes() (New version)
This is a follow up on PR #8984, as the corresponding branch for such PR
was damaged.
You can merge this pull request into a Git repository by
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9559#issuecomment-155247598
**[Test build #45450 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45450/consoleFull)**
for PR 9559 at commit
[`e1d1106`](https://gith
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9227#issuecomment-155247552
A couple of small things left, otherwise 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 pro
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9559#issuecomment-155247427
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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9227#discussion_r44356532
--- Diff:
network/shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleClient.java
---
@@ -137,9 +137,10 @@ public void registerWithShuffleSe
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9237#issuecomment-155247503
> I'm a little worried that disabling SO_REUSEADDR might cause ugly log
messages when you start applications back to back... aside from avoiding
inadvertently running in
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9559#issuecomment-155247407
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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9227#discussion_r44356452
--- Diff:
network/common/src/main/java/org/apache/spark/network/server/TransportChannelHandler.java
---
@@ -109,18 +109,22 @@ public void channelRead0(Channel
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9227#discussion_r44356351
--- Diff:
network/common/src/main/java/org/apache/spark/network/server/TransportChannelHandler.java
---
@@ -55,16 +55,19 @@
private final TransportResp
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9227#discussion_r44356293
--- Diff:
network/common/src/main/java/org/apache/spark/network/TransportContext.java ---
@@ -58,15 +58,24 @@
private final TransportConf conf;
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9559#issuecomment-155247009
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-155246598
**[Test build #45448 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45448/consoleFull)**
for PR 8559 at commit
[`609b61e`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9530#issuecomment-155246626
**[Test build #45446 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45446/consoleFull)**
for PR 9530 at commit
[`a222c03`](https://gith
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9264#issuecomment-155246650
@reggert I've closed your issue as a duplicate. Can you remove it from the
title and use only `SPARK-9026` to avoid potential confusion?
---
If your project is set u
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9264#discussion_r44356014
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Utils.scala
---
@@ -1,393 +0,0 @@
-/*
- * Licensed to th
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44355987
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -177,6 +178,7 @@ object FunctionRegistry {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9565#issuecomment-155246523
**[Test build #45449 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45449/consoleFull)**
for PR 9565 at commit
[`1e13ff9`](https://gith
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44355961
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -108,7 +109,16 @@ trait CheckAnalysis {
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9143#issuecomment-155246265
> Test build #45434 has finished for PR 9143 at commit 1fe2e7b.
>
> This patch fails Spark unit tests.
> This patch merges cleanly.
> This patch adds the
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/9489#issuecomment-155246119
Looks like dply has aggregate functions for variance, standard deviations ?
Can we just match the syntax there ?
https://www.rstudio.com/wp-content/uploads/2015/02/data
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44355776
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -525,21 +526,15 @@ class Analyzer(
case u @ U
Github user sureshthalamati commented on the pull request:
https://github.com/apache/spark/pull/9162#issuecomment-155245870
@rxin Wondering if I need any additional work on this patch ? Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44355675
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/SqlParser.scala ---
@@ -272,7 +273,7 @@ object SqlParser extends AbstractSparkSQLParser with
Github user winningsix commented on the pull request:
https://github.com/apache/spark/pull/8880#issuecomment-155245686
Hi @vanzin , I have updated the patch addressing your comments, do some
code clean ups and add more tests for cryptocodec. Could you help me review it?
Thank you!
-
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9565#issuecomment-155245715
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/9565#issuecomment-155245730
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/9578#issuecomment-155245709
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 pestilence669 opened a pull request:
https://github.com/apache/spark/pull/9578
Fix typo in driver page
"Comamnd property" => "Command property"
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pestilence669/spark fix_sp
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/9565#issuecomment-155245604
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 fea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9428#issuecomment-155245526
**[Test build #45447 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45447/consoleFull)**
for PR 9428 at commit
[`93c8feb`](https://gith
Github user winningsix commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r44355437
--- Diff:
core/src/main/scala/org/apache/spark/crypto/CommonConfigurationKeys.scala ---
@@ -0,0 +1,48 @@
+/*
+ * Licensed to the Apache Software Fo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-155244934
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/9530#issuecomment-155244922
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 zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/9428#discussion_r44355235
--- Diff: core/src/test/scala/org/apache/spark/CheckpointSuite.scala ---
@@ -251,6 +252,46 @@ class CheckpointSuite extends SparkFunSuite with
LocalSparkConte
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9428#issuecomment-155244905
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/9428#issuecomment-155244937
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/8559#issuecomment-155244915
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/9530#issuecomment-155244901
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 yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/9577#issuecomment-155244740
Thanks for merging it!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8639#issuecomment-155244104
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 felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9489#issuecomment-155244366
@mengxr possibly.. though there are usage difference with SparkR
DataFrame/Column as compared to R data.frame (eg. `agg` vs
[`aggregate`](https://stat.ethz.ch/R-man
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9092#issuecomment-155243776
**[Test build #45439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45439/consoleFull)**
for PR 9092 at commit
[`2663cbf`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8639#issuecomment-155243910
**[Test build #45416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45416/consoleFull)**
for PR 8639 at commit
[`785e4ae`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9092#issuecomment-155244180
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 feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9513#issuecomment-155243816
LGTM
If we do decide to change the inheritance structure it should be done
before 1.6 release to prevent breaking public APIs
---
If your project is set u
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9577
---
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 jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/9513#discussion_r44354764
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala ---
@@ -0,0 +1,668 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9577#issuecomment-155242262
Merging with master and branch-1.6
---
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9513#discussion_r44354690
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala ---
@@ -0,0 +1,668 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9513#discussion_r44354638
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala ---
@@ -0,0 +1,668 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9557
---
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/9577#issuecomment-155241916
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9577#issuecomment-155241813
**[Test build #45441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45441/consoleFull)**
for PR 9577 at commit
[`e07f118`](https://git
Github user winningsix commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r44354373
--- Diff:
core/src/test/scala/org/apache/spark/crypto/JceAesCtrCryptoCodecSuite.scala ---
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9557#issuecomment-155241396
**[Test build #2021 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2021/consoleFull)**
for PR 9557 at commit
[`8d10a25`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9575#issuecomment-155241291
**[Test build #45444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45444/consoleFull)**
for PR 9575 at commit
[`828c3b4`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9143#issuecomment-155241173
**[Test build #45445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45445/consoleFull)**
for PR 9143 at commit
[`1fe2e7b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9575#issuecomment-155240995
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/9575#issuecomment-155241014
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/9143#issuecomment-155241026
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/9143#issuecomment-155240999
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/9575#issuecomment-155240441
Jenkins, 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 h
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9562#discussion_r44353866
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1448,4 +1448,21 @@ class SQLQuerySuite extends QueryTest with
Github user brkyvz commented on the pull request:
https://github.com/apache/spark/pull/9143#issuecomment-155240395
Failed to a flaky test. Restarting
---
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 no
Github user brkyvz commented on the pull request:
https://github.com/apache/spark/pull/9143#issuecomment-155240408
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 featur
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9555
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155240352
**[Test build #2026 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2026/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://git
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9562#discussion_r44353792
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala ---
@@ -749,10 +749,14 @@ class DataFrame private[sql](
// will remove intermed
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9546#discussion_r44353658
--- Diff:
core/src/main/scala/org/apache/spark/util/AsynchronousListenerBus.scala ---
@@ -66,6 +66,7 @@ private[spark] abstract class AsynchronousListenerB
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9403#issuecomment-155240080
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9403#issuecomment-155239960
**[Test build #45422 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45422/consoleFull)**
for PR 9403 at commit
[`932485a`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9555#issuecomment-155240029
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9513#issuecomment-155240086
**[Test build #2025 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2025/consoleFull)**
for PR 9513 at commit
[`16a061c`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9555#issuecomment-155239894
**[Test build #45425 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45425/consoleFull)**
for PR 9555 at commit
[`0a5a161`](https://git
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9562#discussion_r44353530
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala ---
@@ -749,10 +749,14 @@ class DataFrame private[sql](
// will remove intermed
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9314
---
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9092#issuecomment-155239682
LGTM pending tests
---
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 f
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9555#issuecomment-155239475
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9143#issuecomment-155239389
**[Test build #45434 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45434/consoleFull)**
for PR 9143 at commit
[`1fe2e7b`](https://git
501 - 600 of 1322 matches
Mail list logo