Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43793910
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43793226
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-153506214
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/9206#issuecomment-153506072
**[Test build #44951 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44951/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-153506212
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/9206#issuecomment-153474396
**[Test build #44951 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44951/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-153472421
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 zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-153467662
LGTM. Just a small nit about the missing doc.
---
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/9206#issuecomment-153472447
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 zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43797572
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152948180
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/9206#issuecomment-152948182
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/9206#issuecomment-152948027
**[Test build #44795 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44795/consoleFull)**
for PR 9206 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-153193676
ping @rxin @zsxwing
This is blocking other stuff I'm working on so I'd really like to push this
soon.
---
If your project is set up for it, you can reply to
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43709051
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/StreamInterceptor.java
---
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43710902
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/StreamInterceptor.java
---
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43710920
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/StreamInterceptor.java
---
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152921811
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/9206#issuecomment-152922663
**[Test build #44795 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44795/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152921824
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/9206#issuecomment-152791764
**[Test build #44742 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44742/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152791784
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/9206#issuecomment-152791785
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/9206#issuecomment-152784110
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/9206#issuecomment-152784108
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152784092
Unrelated failure. 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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152784398
**[Test build #44742 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44742/consoleFull)**
for PR 9206 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43478074
--- Diff:
network/common/src/test/java/org/apache/spark/network/util/TransportFrameDecoderSuite.java
---
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152462600
**[Test build #44672 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44672/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152461532
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/9206#issuecomment-152461553
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/9206#issuecomment-152489749
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/9206#issuecomment-152489343
**[Test build #44672 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44672/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152687246
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/9206#issuecomment-152687416
**[Test build #44724 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44724/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152687242
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/9206#issuecomment-152690739
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/9206#issuecomment-152690734
**[Test build #44724 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44724/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152690740
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43389586
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43398814
--- Diff:
network/common/src/test/java/org/apache/spark/network/util/TransportFrameDecoderSuite.java
---
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43395471
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/TransportResponseHandler.java
---
@@ -50,6 +55,8 @@
private final
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43357714
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152106469
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/9206#issuecomment-152107187
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/9206#issuecomment-152107173
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/9206#issuecomment-152106496
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/9206#issuecomment-152123057
**[Test build #44584 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44584/consoleFull)**
for PR 9206 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152118828
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/9206#issuecomment-152119412
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/9206#issuecomment-152119345
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/9206#issuecomment-152134493
**[Test build #44583 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44583/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152134636
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/9206#issuecomment-152134637
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/9206#issuecomment-152148484
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/9206#issuecomment-152148360
**[Test build #44584 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44584/consoleFull)**
for PR 9206 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152148481
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-151760553
let's try more people, /cc @zsxwing
---
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 squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43277641
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43281238
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/TransportResponseHandler.java
---
@@ -50,6 +55,8 @@
private final
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43282659
--- Diff:
network/common/src/main/java/org/apache/spark/network/server/StreamManager.java
---
@@ -47,6 +47,14 @@
public abstract ManagedBuffer
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43278694
--- Diff:
network/common/src/test/java/org/apache/spark/network/util/TransportFrameDecoderSuite.java
---
@@ -0,0 +1,144 @@
+/*
+ * Licensed to the
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43279805
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/StreamCallback.java
---
@@ -0,0 +1,40 @@
+/*
+ * Licensed to the Apache
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43285564
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43334824
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43335302
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/TransportResponseHandler.java
---
@@ -50,6 +55,8 @@
private final
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43335526
--- Diff:
network/common/src/main/java/org/apache/spark/network/util/TransportFrameDecoder.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43335096
--- Diff:
network/common/src/main/java/org/apache/spark/network/server/StreamManager.java
---
@@ -47,6 +47,14 @@
public abstract ManagedBuffer
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43335377
--- Diff:
network/common/src/main/java/org/apache/spark/network/client/StreamCallback.java
---
@@ -0,0 +1,40 @@
+/*
+ * Licensed to the Apache
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-150076974
**[Test build #44102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44102/consoleFull)**
for PR 9206 at commit
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/9206
[SPARK-11235] [network] Add ability to stream data using network lib.
The current interface used to fetch shuffle data is not very efficient for
large buffers; it requires the receiver to buffer
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-150056546
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/9206#issuecomment-150057544
**[Test build #44102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44102/consoleFull)**
for PR 9206 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-150058459
/cc @rxin @aarondav
---
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/9206#issuecomment-150056518
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
75 matches
Mail list logo