[SPARK-8482] Added M4 instances to the list.

[pradeepchhetri]

AWS recently added M4 instances (https://aws.amazon.com/blogs/aws/the-new-m4-instance-type-bonus-price-reduction-on-m3-c4/).

[srowen]

https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark
Usually we ask for JIRAs

[pradeepchhetri]

@srowen I created a JIRA ticket and updated the title with that.

[JoshRosen]

We might as well include the new T2.large instance while we're at it: https://aws.amazon.com/blogs/aws/new-t2-large-instances/

[pradeepchhetri]

@JoshRosen Thank you for reminding it. Added t2.large as well.

[JoshRosen]

Jenkins, this is ok to test.

(So the Python style checks are run)

[JoshRosen]

Jenkins, this is ok to test.

[SparkQA]

Test build #35354 has finished for PR 6899 at commit 4f4ea79.

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

[JoshRosen]

Jenkins, retest this please.

[JoshRosen]

(Retesting to test a build script change)

[SparkQA]

Test build #35374 has finished for PR 6899 at commit 4f4ea79.

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

[AmplabJenkins]

Merged build finished. Test FAILed.

[JoshRosen]

@davies @ahirreddy The tests here passed, but the test-selection logic was too good: by skipping all of the Scala / Java tests, we didn't write any JUnit XML files, causing Jenkins to mark the build as failed. Maybe we should just add a dummy XML file to the build / repo to avoid this (or see if we can somehow configure the Jenkins plugin to be more permissive w.r.t missing report files (the artifact upload plugin is robust against this, for example)).

[srowen]

LGTM. This is safe to commit in that there was no actual test failure due to this right?

[JoshRosen]

Yeah it should be fine to commit; go ahead.

Sent from my phone

On Jun 22, 2015, at 1:25 AM, Sean Owen [email protected] wrote:

LGTM. This is safe to commit in that there was no actual test failure due to this right?

—
Reply to this email directly or view it on GitHub.

[shivaram]

Merging this