Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Sep 1, 2015

Migrate Apache download closer.cgi refs to new closer.lua

This is the bit of the change that affects the project docs; I'm implementing the changes to the Apache site separately.

@lresende
Copy link
Member

lresende commented Sep 1, 2015

LGTM. I have also closed my PR.

@JoshRosen
Copy link
Contributor

LGTM.

@SparkQA
Copy link

SparkQA commented Sep 1, 2015

Test build #41872 has finished for PR 8557 at commit f0adacd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member Author

srowen commented Sep 1, 2015

The failures must be spurious. One is a doc-only change; the other affects a docker script that isn't involved in the tests here. (Actually I'm not sure that download link would work as it's used there anyway, but a different story). Merging to 1.5 and master

asfgit pushed a commit that referenced this pull request Sep 1, 2015
…ring scripts

Migrate Apache download closer.cgi refs to new closer.lua

This is the bit of the change that affects the project docs; I'm implementing the changes to the Apache site separately.

Author: Sean Owen <[email protected]>

Closes #8557 from srowen/SPARK-10398.

(cherry picked from commit 3f63bd6)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in 3f63bd6 Sep 1, 2015
@srowen srowen deleted the SPARK-10398 branch September 2, 2015 10:55
@nchammas
Copy link
Contributor

nchammas commented Nov 2, 2015

(Actually I'm not sure that download link would work as it's used there anyway, but a different story).

FYI: It's not working for me. I think it has to do with Apache's Lua script, though.

Related discussion on spark-dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants