Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8383
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/8579
---
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 davies commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-139049449
@andrewor14 Is this ready to go?
---
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/8579#issuecomment-138756982
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138756976
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/8579#issuecomment-138756687
[Test build #42166 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42166/console)
for PR 8579 at commit
[`c861d72`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138734763
[Test build #42166 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42166/consoleFull)
for PR 8579 at commit
[`c861d72`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138734619
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/8579#issuecomment-138734662
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138733977
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/8579#issuecomment-138732951
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138732898
[Test build #42154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42154/console)
for PR 8579 at commit
[`c861d72`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138732952
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138711081
LGTM. Maybe @JoshRosen would like to take a pass before this gets merged?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138706596
[Test build #42154 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42154/consoleFull)
for PR 8579 at commit
[`c861d72`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138703283
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/8579#issuecomment-138703225
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 davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38980901
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138699177
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/8579#issuecomment-138699178
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138699033
[Test build #42144 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42144/console)
for PR 8579 at commit
[`aff34ee`](https://github.
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138692449
@davies I looked at this patch in great detail and believe it's correct and
very efficient. This looks good pending a few comments and readability
improvements I sugg
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975999
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38976588
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/joins/OuterJoinSuite.scala
---
@@ -97,17 +97,17 @@ class OuterJoinSuite extends SparkPlan
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975823
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975863
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975669
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975742
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975588
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975477
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975349
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975215
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38975270
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,189 @@ private class RightOuterIterator
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138659722
[Test build #42144 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42144/consoleFull)
for PR 8579 at commit
[`aff34ee`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-138656903
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/8579#issuecomment-138656857
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137910501
[Test build #42035 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42035/console)
for PR 8579 at commit
[`ce57f67`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137910504
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137910503
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137910372
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137910374
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137910275
[Test build #42035 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42035/consoleFull)
for PR 8579 at commit
[`ce57f67`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137909760
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/8579#issuecomment-137909758
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/8579#issuecomment-137909616
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/8579#issuecomment-137909582
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 andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38788635
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,203 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38785328
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -71,6 +75,8 @@ case class SortMergeOuterJoin(
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38784898
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +297,203 @@ private class RightOuterIterator
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38784804
--- Diff: core/src/main/scala/org/apache/spark/util/collection/BitSet.scala
---
@@ -33,6 +33,17 @@ class BitSet(numBits: Int) extends Serializable {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38708955
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -132,15 +132,10 @@ private[sql] abstract class SparkStrategies ex
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8579#discussion_r38708327
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -132,15 +132,10 @@ private[sql] abstract class SparkStrategie
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137350715
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137350711
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/8383#issuecomment-137350369
[Test build #41958 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41958/console)
for PR 8383 at commit
[`62cc7e3`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137328959
[Test build #41958 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41958/consoleFull)
for PR 8383 at commit
[`62cc7e3`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137328168
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/8383#issuecomment-137328114
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137327156
@davies Thanks for comment and continuing this patch. I still added new
commits to fix the problems you mentioned. But free feel to merge #8579 when it
is ready. I will c
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137287379
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/8579#issuecomment-137287380
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137287302
[Test build #41948 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41948/console)
for PR 8579 at commit
[`16756a6`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137277982
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137277936
[Test build #41946 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41946/console)
for PR 8579 at commit
[`8a81df4`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137277981
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137260752
[Test build #41948 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41948/consoleFull)
for PR 8579 at commit
[`16756a6`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137259513
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/8579#issuecomment-137259497
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-13725
[Test build #41946 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41946/consoleFull)
for PR 8579 at commit
[`8a81df4`](https://gith
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r38589660
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -71,6 +77,7 @@ case class SortMergeOuterJoin(
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137255108
@viirya I had created #8579. If you'd like to continue to work on this one,
feel free to pull that one into this one.
---
If your project is set up for it, you can reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8579#issuecomment-137254994
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/8579#issuecomment-137255024
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 davies opened a pull request:
https://github.com/apache/spark/pull/8579
[SPARK-9730] [SQL] Add Full Outer Join support for SortMergeJoin
This PR is based on #8383 , thanks to @viirya
JIRA: https://issues.apache.org/jira/browse/SPARK-9730
This patch adds
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r38589087
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +298,224 @@ private class RightOuterIterator(
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137207810
@viirya Thanks for working on this, this patch is still over complicated,
do you mind me to take over it?
---
If your project is set up for it, you can reply to this ema
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r38562684
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +298,224 @@ private class RightOuterIterator(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137183668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137183663
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/8383#issuecomment-137183448
[Test build #41936 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41936/console)
for PR 8383 at commit
[`addf5fe`](https://github.
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r38561395
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +298,224 @@ private class RightOuterIterator(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137136877
[Test build #41936 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41936/consoleFull)
for PR 8383 at commit
[`addf5fe`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-137134840
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/8383#issuecomment-137134797
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r38546234
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +300,243 @@ private class RightOuterIterator(
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r38472819
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeOuterJoin.scala
---
@@ -271,3 +300,243 @@ private class RightOuterIterator(
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-136850070
@davies @JoshRosen
---
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/8383#issuecomment-134871683
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134871682
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/8383#issuecomment-134871447
[Test build #41595 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41595/console)
for PR 8383 at commit
[`27b6044`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134831513
[Test build #41595 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41595/consoleFull)
for PR 8383 at commit
[`27b6044`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134830984
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/8383#issuecomment-134830972
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/8383#issuecomment-134713311
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134713306
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134713141
[Test build #41538 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41538/console)
for PR 8383 at commit
[`0559a40`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134680871
[Test build #41538 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41538/consoleFull)
for PR 8383 at commit
[`0559a40`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8383#issuecomment-134679121
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/8383#issuecomment-134679079
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/8383#discussion_r37888287
--- Diff:
unsafe/src/main/java/org/apache/spark/unsafe/bitset/BitSetMethods.java ---
@@ -68,6 +68,19 @@ public static boolean isSet(Object baseObject, long
ba
1 - 100 of 159 matches
Mail list logo