Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8005
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128613097
Merging in master & branch-1.5.
---
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/8005#issuecomment-128612972
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/8005#issuecomment-128612932
[Test build #40138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40138/console)
for PR 8005 at commit
[`f9c4aff`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128585593
[Test build #40138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40138/consoleFull)
for PR 8005 at commit
[`f9c4aff`](https://gith
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128584909
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128583983
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/8005#issuecomment-128583941
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 a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36484034
--- Diff:
core/src/test/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleWriterSuite.java
---
@@ -474,62 +478,22 @@ public void
writeRecordsThatAreBiggerThanD
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483853
--- Diff:
core/src/test/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleWriterSuite.java
---
@@ -234,7 +238,7 @@ private void assertSpillFilesWereCleanedUp()
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483837
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -397,53 +399,84 @@ private void allocateSpaceForRecord(int
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483832
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -397,53 +399,84 @@ private void allocateSpaceForRecord(int
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483774
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -123,10 +126,10 @@ public UnsafeShuffleExternalSorter(
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483772
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -123,10 +126,10 @@ public UnsafeShuffleExternalSorter(
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483768
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -87,11 +90,11 @@
private final LinkedList spil
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483757
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -34,14 +34,17 @@
import org.apache.spark.serializer.Du
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8005#discussion_r36483761
--- Diff:
core/src/main/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleExternalSorter.java
---
@@ -52,7 +55,7 @@
* written as a single serialized, comp
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128544990
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/8005#issuecomment-128544781
[Test build #40076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40076/console)
for PR 8005 at commit
[`9de5c72`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128516357
[Test build #40076 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40076/consoleFull)
for PR 8005 at commit
[`9de5c72`](https://gith
GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/8005
[SPARK-9453] [SQL] support records larger than page size in
UnsafeShuffleExternalSorter
This patch follows exactly #7891 (except testing)
You can merge this pull request into a Git repository by run
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8005#issuecomment-128515314
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/8005#issuecomment-128515292
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
23 matches
Mail list logo