Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-132235364
@zhzhan thanks for your help; as it turns out, I'm pretty sure my guess was
correct because making those changes fixed the test locally. Still need to
check it out on Jen
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-132170089
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/8033#issuecomment-132170083
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/8033#issuecomment-132169942
[Test build #41118 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41118/console)
for PR 8033 at commit
[`17e443e`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-132136261
[Test build #41118 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41118/consoleFull)
for PR 8033 at commit
[`17e443e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-132132992
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/8033#issuecomment-132133111
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 zhzhan commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131960610
@srowen Probably you can revert back the change in
sql/hive/src/main/scala/org/apache/spark/sql/hive/orc/OrcRelation.scala
---
If your project is set up for it, you can
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131960551
@zhzhan if you have a moment to look I'd be grateful. I am a little stuck.
I suspect these lines at the moment:
https://github.com/apache/spark/pull/8033/files#diff-287b5
Github user zhzhan commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131959546
@srowen It seems that the mapping got messed up, which I don't have clue
yet and didn't find any obvious reason why the patch can break the test. I will
dig more and let
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131945578
@zhzhan could I get your eyes on this change? My change is causing
`OrcQuerySuite` to fail, and I'm not yet clear why. I see you created much of
this test. I'm looking fo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131895473
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/8033#issuecomment-131895471
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/8033#issuecomment-131895278
[Test build #41013 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41013/console)
for PR 8033 at commit
[`e772d9f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131847860
[Test build #41013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41013/consoleFull)
for PR 8033 at commit
[`e772d9f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131844798
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/8033#issuecomment-131733920
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/8033#issuecomment-131097690
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/8033#issuecomment-131097507
[Test build #40863 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40863/console)
for PR 8033 at commit
[`14c912f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131061540
[Test build #40863 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40863/consoleFull)
for PR 8033 at commit
[`14c912f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-131061413
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/8033#issuecomment-131061424
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/8033#issuecomment-130817168
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/8033#issuecomment-130817052
[Test build #40781 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40781/console)
for PR 8033 at commit
[`b57657c`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130765202
[Test build #40781 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40781/consoleFull)
for PR 8033 at commit
[`b57657c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130764425
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/8033#issuecomment-130764490
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/8033#issuecomment-130324603
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/8033#issuecomment-130322961
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/8033#issuecomment-130322991
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/8033#issuecomment-130271461
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 does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130271362
[Test build #40617 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40617/console)
for PR 8033 at commit
[`03b747b`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130264265
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/8033#issuecomment-130264226
[Test build #40618 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40618/console)
for PR 8033 at commit
[`7c92505`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130242794
[Test build #40618 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40618/consoleFull)
for PR 8033 at commit
[`7c92505`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130241758
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/8033#issuecomment-130241691
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/8033#issuecomment-130238768
[Test build #40617 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40617/consoleFull)
for PR 8033 at commit
[`03b747b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130238218
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 have thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130238264
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 this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130089835
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/8033#issuecomment-130089829
[Test build #40518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40518/console)
for PR 8033 at commit
[`5815596`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130088956
[Test build #40518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40518/consoleFull)
for PR 8033 at commit
[`5815596`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-130088311
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/8033#issuecomment-130088255
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8033#discussion_r36771152
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala
---
@@ -38,7 +38,7 @@ object LogisticRegressionSuite {
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/8033#discussion_r36768472
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala
---
@@ -38,7 +38,7 @@ object LogisticRegressionSuite {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129911818
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/8033#issuecomment-129911671
[Test build #40456 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40456/console)
for PR 8033 at commit
[`08ff887`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129890844
[Test build #40456 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40456/consoleFull)
for PR 8033 at commit
[`08ff887`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129889448
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/8033#issuecomment-129889294
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/8033#issuecomment-129852262
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/8033#issuecomment-129850958
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/8033#issuecomment-129851002
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/8033#issuecomment-129633610
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/8033#issuecomment-129633579
[Test build #40332 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40332/console)
for PR 8033 at commit
[`f8f8898`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129622319
[Test build #40332 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40332/consoleFull)
for PR 8033 at commit
[`f8f8898`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129621779
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/8033#issuecomment-129621741
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129595251
Nice one, since that's what I've been doing. Yes it's clear that this will
end up creating some more verbose code, but one where the conversions are
visible. I personally
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129592820
> Is the point to get rid of implicits only, or also replace
JavaConversions with JavaConverters? I wasn't sure whether the latter is
preferred.
The point is
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129591772
[Test build #40311 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40311/console)
for PR 8033 at commit
[`a353317`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129591775
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/8033#issuecomment-129588203
[Test build #40311 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40311/consoleFull)
for PR 8033 at commit
[`a353317`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129585457
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/8033#issuecomment-129585355
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/8033#issuecomment-129527147
[Test build #40291 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40291/console)
for PR 8033 at commit
[`d82a730`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129527150
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/8033#issuecomment-129525981
[Test build #40291 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40291/consoleFull)
for PR 8033 at commit
[`d82a730`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-129525259
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/8033#issuecomment-129434288
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/8033#issuecomment-128783359
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/8033#issuecomment-128783145
[Test build #40172 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40172/console)
for PR 8033 at commit
[`d2aed5a`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-128738026
[Test build #40172 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40172/consoleFull)
for PR 8033 at commit
[`d2aed5a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8033#issuecomment-128734574
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/8033#issuecomment-128734703
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 srowen opened a pull request:
https://github.com/apache/spark/pull/8033
[SPARK-9613] [CORE] [WIP] Ban use of JavaConversions and migrate all
existing uses to JavaConverters
Replace JavaConversions implicits with JavaConverters (WIP)
This is only ~20% of the occu
78 matches
Mail list logo