Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1330
---
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55352303
Ok, I am merging this over the alternative PR #1323, which is now closed.
Thanks to everyone who looked at this.
---
If your project is set up for it, you can reply t
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55312782
@witgo Sorry, I had not realized that this had not been updated since the
discussions. Just tested it, and it worked for me. LGTM
---
If your project is set up for
Github user witgo commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55236678
No postfix warnings in 179ba61 .
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55236013
The net-net change for all of these overlapping PRs should be: remove the
compiler flag, and, add the import everywhere that the warning occurs. If there
are warnings afte
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55216430
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/20134/consoleFull)
for PR 1330 at commit
[`179ba61`](https://github.com/a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55213209
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/20134/consoleFull)
for PR 1330 at commit
[`179ba61`](https://github.com/ap
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/1330#discussion_r17401380
--- Diff: pom.xml ---
@@ -839,7 +839,6 @@
-unchecked
-deprecation
-feature
- -language:
Github user witgo commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55210927
The code has been updated.
---
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-55198371
@witgo Hi, I was asked to take a look at this PR. I tested the current
master vs. this PR merged with the master, and I actually found that this PR
added postfix war
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-54315691
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19665/consoleFull)
for PR 1330 at commit
[`15d5ed3`](https://github.com/a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-54306268
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19665/consoleFull)
for PR 1330 at commit
[`15d5ed3`](https://github.com/ap
Github user witgo commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-53365696
@andrewor14 , @srowen
This is mainly to solve the problem of importing the
scala.language.postfixOps and org.scalatest.time.SpanSugar._ at the same time.
---
If yo
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-53340913
This is a sort of duplicate of https://github.com/apache/spark/pull/1069
@witgo this was opened a few times?
From the discussion, I am not clear this should be com
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-53340219
test 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 fea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-51078602
QA results for PR 1330:- This patch PASSES unit tests.- This patch
merges cleanly- This patch adds no public classesFor more
information see test
ouptut:https://amplab.c
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-51070559
QA tests have started for PR 1330. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17855/consoleFull
---
If
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49122651
QA results for PR 1330:- This patch PASSES unit tests.- This patch
merges cleanly- This patch adds no public classesFor more
information see test
ouptut:https://amplab.c
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49117911
QA tests have started for PR 1330. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16702/consoleFull
---
If
Github user witgo commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49117596
As a result of #772. The master has fixed this problem. But we should
remove this line `-language:postfixOps` in
[pom.xml#L807](https://github.com/apache/spark/blob/maste
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49105352
Could you paste exactly which warnings you are eliminating? I don't see any
warnings in our master jenkins build that seem relevant to these changes.
---
If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49085897
QA tests have started for PR 1330. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16690/consoleFull
---
If
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49085313
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 hav
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1330#issuecomment-49085302
Jenkins, retest this pleae.
---
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
24 matches
Mail list logo