Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198177508
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 projec
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56576152
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197573849
@rxin, I believe that I've addressed all of your review comments (plus the
outstanding TODOs that I left).
---
If your project is set up for it, you can reply to thi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198007459
**[Test build #53442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53442/consoleFull)**
for PR 11748 at commit
[`3fc0b66`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198007480
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197579760
**[Test build #53355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53355/consoleFull)**
for PR 11748 at commit
[`719ad3c`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198177377
**[Test build #53489 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53489/consoleFull)**
for PR 11748 at commit
[`2970932`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198147634
**[Test build #53489 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53489/consoleFull)**
for PR 11748 at commit
[`2970932`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197603358
**[Test build #53351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53351/consoleFull)**
for PR 11748 at commit
[`4f5074e`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198081291
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 projec
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198030435
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198013392
**[Test build #53445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53445/consoleFull)**
for PR 11748 at commit
[`cb9311b`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197579877
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56576372
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198013402
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 projec
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56576574
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197579872
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 projec
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56576488
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Founda
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197602022
**[Test build #53352 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53352/consoleFull)**
for PR 11748 at commit
[`b6ddf3e`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197603929
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198004374
**[Test build #53442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53442/consoleFull)**
for PR 11748 at commit
[`3fc0b66`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197620299
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 projec
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198003076
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197586228
Looks good from my side. Would be good for somebody else to take a look at
this too. E.g. @andrewor14 ?
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198009919
**[Test build #53445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53445/consoleFull)**
for PR 11748 at commit
[`cb9311b`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197603931
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197553065
**[Test build #53352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53352/consoleFull)**
for PR 11748 at commit
[`b6ddf3e`](https://gi
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56575880
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198178084
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198080899
**[Test build #53452 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53452/consoleFull)**
for PR 11748 at commit
[`cb9311b`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11748
---
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 ena
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56423712
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskResultGetter.scala ---
@@ -81,8 +81,9 @@ private[spark] class TaskResultGetter(sparkEnv: Spark
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197620300
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197574132
**[Test build #53363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53363/consoleFull)**
for PR 11748 at commit
[`3fc0b66`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198081295
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56576121
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Founda
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198177509
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198032510
**[Test build #53452 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53452/consoleFull)**
for PR 11748 at commit
[`cb9311b`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197602624
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198013408
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197551394
**[Test build #53351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53351/consoleFull)**
for PR 11748 at commit
[`4f5074e`](https://gi
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56576904
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Founda
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197620123
**[Test build #53363 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53363/consoleFull)**
for PR 11748 at commit
[`3fc0b66`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197556313
**[Test build #53355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53355/consoleFull)**
for PR 11748 at commit
[`719ad3c`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197602631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198007472
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197218055
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197218058
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197217818
**[Test build #53290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53290/consoleFull)**
for PR 11748 at commit
[`0a347fd`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197184698
**[Test build #53290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53290/consoleFull)**
for PR 11748 at commit
[`0a347fd`](https://gi
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284643
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskResultGetter.scala ---
@@ -81,8 +81,9 @@ private[spark] class TaskResultGetter(sparkEnv: Spark
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284364
--- Diff:
core/src/test/scala/org/apache/spark/io/ChunkedByteBufferSuite.scala ---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284334
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Founda
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284289
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskResultGetter.scala ---
@@ -81,8 +81,9 @@ private[spark] class TaskResultGetter(sparkEnv: SparkEnv,
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284273
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskResultGetter.scala ---
@@ -81,8 +81,9 @@ private[spark] class TaskResultGetter(sparkEnv: SparkEnv,
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284277
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Founda
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284073
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56284010
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56283999
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56283991
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56283895
--- Diff:
core/src/test/scala/org/apache/spark/io/ChunkedByteBufferSuite.scala ---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11748#discussion_r56283941
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -0,0 +1,143 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197115612
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197115615
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197115420
**[Test build #53251 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53251/consoleFull)**
for PR 11748 at commit
[`e5e663f`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-197090249
**[Test build #53251 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53251/consoleFull)**
for PR 11748 at commit
[`e5e663f`](https://gi
66 matches
Mail list logo
