Skip to content

Conversation

@koaning
Copy link
Contributor

@koaning koaning commented Jun 28, 2015

This would otherwise need to be set manually by R users in AWS.

https://issues.apache.org/jira/browse/SPARK-8596

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@shivaram
Copy link
Contributor

Jenkins, add to whitelist

@shivaram
Copy link
Contributor

Jenkins, ok to test

ec2/spark_ec2.py Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really minor but could you add a comment on the line before saying this is for RStudio ? Might help others figure out why this port is open

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@SparkQA
Copy link

SparkQA commented Jun 28, 2015

Test build #35945 has started for PR 7068 at commit ce6ad88.

@SparkQA
Copy link

SparkQA commented Jun 28, 2015

Test build #35945 has finished for PR 7068 at commit ce6ad88.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@koaning
Copy link
Contributor Author

koaning commented Jun 28, 2015

@shivaram. Comment might make sense (I didn't add because most ports didn't have comments).

Comment added.

@SparkQA
Copy link

SparkQA commented Jun 28, 2015

Test build #35946 has started for PR 7068 at commit ac8100d.

@SparkQA
Copy link

SparkQA commented Jun 28, 2015

Test build #35946 has finished for PR 7068 at commit ac8100d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@koaning
Copy link
Contributor Author

koaning commented Jun 28, 2015

just out of curiosity. what is making this test fail?

@shivaram
Copy link
Contributor

The tests are not actually failing. The message from SparkQA is the one to use (it says passes all tests). There is a bug in the AmplabJenkins messages that @JoshRosen is looking into.

LGTM. Thanks @koaning for the patch. Merging this

@asfgit asfgit closed this in 9ce78b4 Jun 28, 2015
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.

4 participants