Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8209
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131264579
[Test build #1607 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1607/console)
for PR 8209 at commit
[`9111dc0`](https://github
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131230888
[Test build #1607 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1607/consoleFull)
for PR 8209 at commit
[`9111dc0`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131229840
[Test build #40896 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40896/console)
for PR 8209 at commit
[`4a743c2`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-13122
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131229994
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 hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131210873
I'd rather leave the scala source alone for now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131206852
Hm, still seems better for consistency and/or to future-proof; to the
extent the docs are read as HTML it isn't valid. I don't feel super strongly
about addressing it now
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131206616
I have replaced `` tags by `` in all java files I could find
them in. I haven't touched the ```BytesToByteMap```, ```Bin``` and
```PagedTable``` classes becau
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8209#discussion_r37103923
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -211,10 +211,10 @@ public SparkLauncher addSparkArg(String arg) {
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131198780
I'll change those as well. It is strange that these didn't create problems;
I guess it has something to do with the position of the << in the line.
---
If your proje
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/8209#discussion_r37103311
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -211,10 +211,10 @@ public SparkLauncher addSparkArg(String arg) {
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131195350
I found several instances of a similar javadoc problem in:
- BytesToBytesMap
- Bin
- PagedTable
Let's clean those up along the way?
---
If your
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8209#discussion_r37101619
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -211,10 +211,10 @@ public SparkLauncher addSparkArg(String arg) {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131193368
[Test build #40896 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40896/consoleFull)
for PR 8209 at commit
[`4a743c2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131193050
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/8209#issuecomment-131193079
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131192668
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 feature
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8209#issuecomment-131191189
Can one of the admins verify this patch?
---
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 pr
GitHub user hvanhovell opened a pull request:
https://github.com/apache/spark/pull/8209
[SPARK-9980][BUILD] Fix SBT publishLocal error due to invalid characters in
doc
Tiny modification to a few comments ```sbt publishLocal``` work again.
You can merge this pull request into a Git
20 matches
Mail list logo