Skip to content
This repository has been archived by the owner on Jan 9, 2020. It is now read-only.

Exclude flaky ExternalShuffleServiceSuite from Travis #185

Conversation

kimoonkim
Copy link
Member

To help #178 get in shape.

cc @foxish

@foxish foxish mentioned this pull request Mar 14, 2017
@foxish
Copy link
Member

foxish commented Mar 14, 2017

rerun integration test please

@foxish
Copy link
Member

foxish commented Mar 14, 2017

@kimoonkim, looks like one of the travis builds exited with memory errors again.

@cvpatel
Copy link
Member

cvpatel commented Mar 15, 2017

It seem unit-tests are passing now, but there is still a checkstyle failure

@cvpatel
Copy link
Member

cvpatel commented Mar 15, 2017

rerun integration test please

@ash211
Copy link

ash211 commented Mar 15, 2017

Checkstyle failure was fixed upstream in apache@ed84cd06841

@ash211
Copy link

ash211 commented Mar 15, 2017

Will merge this as an incremental improvement and open a new PR cherry picking that commit

Edit: #187

@ash211 ash211 merged commit 2382ea6 into apache-spark-on-k8s:prep-for-alpha-release Mar 15, 2017
@kimoonkim kimoonkim deleted the exclude-flaky-externalshufflesuite branch March 28, 2017 17:45
ifilonenko pushed a commit to ifilonenko/spark that referenced this pull request Feb 25, 2019
…on-k8s#185)

* Fix token leak when logging in CondaEnvironmentManager, and allow configuration of higher priority pkgs_dirs through the conf

* copy yarn integration test logdirs to CIRCLE_ARTIFACTS otherwise we have no idea why they failed

* oops fix envs_dirs, should be a list

* only copy yarn test artifacts if they exist

* argh syntax

* omg bash you killin me
puneetloya pushed a commit to puneetloya/spark that referenced this pull request Mar 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants