Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7925#issuecomment-127882225
LGTM (not super familiar with decimals though)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7904#issuecomment-127883162
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7904#issuecomment-127883122
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7802#issuecomment-127883375
@JDrit what's still WIP about 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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7802#issuecomment-127883935
Actually I think the current API breaks binary compatibility for data
sources, so we can't merge it as is.
In Java (or Scala binary compatibility),
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7925#issuecomment-127884345
[Test build #1344 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1344/console)
for PR 7925 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7904#issuecomment-127884531
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7904#issuecomment-127884542
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7925
---
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 the pull request:
https://github.com/apache/spark/pull/7925#issuecomment-127884616
Merged into master and 1.5 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 your project does
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272160
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorter.java
---
@@ -191,24 +191,28 @@ public void spill() throws
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7774#issuecomment-127884863
[Test build #1349 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1349/consoleFull)
for PR 7774 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272364
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillMerger.java
---
@@ -47,11 +47,19 @@ public int
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272385
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/UnsafeKVExternalSorter.java
---
@@ -82,8 +82,15 @@ public
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272417
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/UnsafeKVExternalSorter.java
---
@@ -82,8 +82,15 @@ public
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7904#issuecomment-127887018
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/6817#issuecomment-127886604
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6817#issuecomment-127886729
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7774#issuecomment-127887044
[Test build #39828 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39828/consoleFull)
for PR 7774 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7774#issuecomment-127886731
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7774#issuecomment-127886593
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
Github user XuTingjun commented on the pull request:
https://github.com/apache/spark/pull/6817#issuecomment-127886475
@squito, I have updated the test, thank you very much.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6817#issuecomment-127888097
[Test build #39827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39827/consoleFull)
for PR 6817 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272567
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/UnsafeFixedWidthAggregationMapSuite.scala
---
@@ -231,4 +231,109 @@ class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7904#issuecomment-127888915
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 nraychaudhuri commented on a diff in the pull request:
https://github.com/apache/spark/pull/7796#discussion_r36272684
--- Diff:
external/kafka/src/test/scala/org/apache/spark/streaming/kafka/DirectKafkaStreamSuite.scala
---
@@ -381,3 +447,20 @@ object
Github user nraychaudhuri commented on a diff in the pull request:
https://github.com/apache/spark/pull/7796#discussion_r36272732
--- Diff:
external/kafka/src/test/scala/org/apache/spark/streaming/kafka/DirectKafkaStreamSuite.scala
---
@@ -381,3 +447,20 @@ object
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272769
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/UnsafeFixedWidthAggregationMapSuite.scala
---
@@ -231,4 +231,109 @@ class
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/7929#issuecomment-127889147
LGTM - feel free to merge, as it is really taking a toll on our tests right
now.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/7948#issuecomment-127889224
Changes look good overall; just one minor comment RE: a typo in a variable
name, plus a comment on tests.
---
If your project is set up for it, you can reply to this
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7929#discussion_r36272840
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -62,6 +64,52 @@ private[hive] class ClientWrapper(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7952#issuecomment-127889358
[Test build #39824 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39824/console)
for PR 7952 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7948#discussion_r36272873
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorter.java
---
@@ -191,24 +191,28 @@ public void spill() throws
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7952#issuecomment-127889452
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 pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/7929#discussion_r36273191
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -62,6 +64,52 @@ private[hive] class ClientWrapper(
GitHub user yjshen opened a pull request:
https://github.com/apache/spark/pull/7953
[SPARK-9628][SQL]Rename int to SQLDate, long to SQLTimestamp for better
readability
JIRA: https://issues.apache.org/jira/browse/SPARK-9628
You can merge this pull request into a Git repository by
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7833#issuecomment-127890431
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7833#issuecomment-127890533
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7833#issuecomment-127890489
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
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/7929#issuecomment-127890665
do feel free to get the comment thing hashed out with @srowen. My time zone
is approaching bed time, so I have to sign off. Would be nice to get something
of this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7833#issuecomment-127890739
[Test build #223 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/223/consoleFull)
for PR 7833 at commit
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/7924#issuecomment-127890833
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7953#issuecomment-127890897
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7924#issuecomment-127890916
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
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127890943
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7953#issuecomment-127890869
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7833#issuecomment-127890915
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7953#issuecomment-127890921
LGTM pending Jenkins passing.
---
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/7833#issuecomment-127890876
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7833#issuecomment-127891028
[Test build #39831 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39831/consoleFull)
for PR 7833 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7924#issuecomment-127891935
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7924#issuecomment-127891878
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127891747
[Test build #1350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1350/consoleFull)
for PR 7893 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127891887
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127891934
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7924#issuecomment-127892111
[Test build #224 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/224/consoleFull)
for PR 7924 at commit
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/7944#issuecomment-127891984
Nah - not a big enough thing to deal to create a new JIRA. Anyways this
LGTM. @JoshRosen feel free to merge.
---
If your project is set up for it, you can reply to
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127892005
[Test build #225 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/225/consoleFull)
for PR 7893 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127892327
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7953#issuecomment-127892353
[Test build #39829 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39829/consoleFull)
for PR 7953 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7761#issuecomment-127892341
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7761#issuecomment-127892387
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127892312
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127892357
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127892342
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
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/7410#issuecomment-127892684
Hi @tdas, would you please help to review this patch, thanks a lot.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user XuTingjun commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127892711
Thanks all, I have added the document .
---
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/7924#issuecomment-127892586
[Test build #39830 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39830/consoleFull)
for PR 7924 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127894040
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
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/7929#discussion_r36273851
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -62,6 +64,52 @@ private[hive] class ClientWrapper(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127894096
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7893#issuecomment-127894190
[Test build #39833 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39833/consoleFull)
for PR 7893 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7761#issuecomment-127894409
[Test build #39834 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39834/consoleFull)
for PR 7761 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127894609
[Test build #39835 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39835/consoleFull)
for PR 7918 at commit
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/7929#discussion_r36274062
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -62,6 +64,52 @@ private[hive] class ClientWrapper(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7918#issuecomment-127895925
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 WangTaoTheTonic commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127896125
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127896314
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127896341
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7948#issuecomment-127896504
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7948#issuecomment-127896521
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127896515
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127896444
[Test build #226 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/226/consoleFull)
for PR 7931 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127896523
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7948#issuecomment-127896823
[Test build #39836 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39836/consoleFull)
for PR 7948 at commit
GitHub user yhuai opened a pull request:
https://github.com/apache/spark/pull/7954
[SPARK-9630] [SQL] [WIP] Clean up new aggregate operators (SPARK-9240
follow up)
This is the followup of https://github.com/apache/spark/pull/7813. It
renames `HybridUnsafeAggregationIterator` to
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127897182
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
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/7825#issuecomment-127897159
Thanks @zsxwing very much for reviewing this. Merging this to master and
1.5.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127897204
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7931#issuecomment-127897258
[Test build #39837 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39837/consoleFull)
for PR 7931 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127897858
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127897877
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
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/7954#discussion_r36275154
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/UnsafeRowSerializer.scala
---
@@ -58,11 +58,26 @@ private class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127898130
[Test build #39839 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39839/consoleFull)
for PR 7954 at commit
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127898231
I will add proper tests to for our fallback strategy.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7913#discussion_r36275319
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/receiver/BlockGenerator.scala
---
@@ -92,29 +95,43 @@ private[streaming] class BlockGenerator(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7825
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127898490
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7954#issuecomment-127898510
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
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/7676#discussion_r36275502
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -76,6 +78,7 @@ case class SortPrefix(child: SortOrder)
1 - 100 of 1470 matches
Mail list logo