New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-13203] Add scalastyle rule banning use of mutable.SynchronizedBuffer #11134

Closed

tedyu wants to merge 1 commit into apache:master from tedyu:master

Contributor

tedyu commented Feb 9, 2016

@andrewor14
Please take a look


          [SPARK-13203] Add scalastyle rule banning use of mutable.Synchronized…

1c80127

…Buffer

Contributor Author

tedyu commented Feb 9, 2016

Jenkins, test this please

SparkQA commented Feb 9, 2016

Test build #50972 has finished for PR 11134 at commit 1c80127.

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

SparkQA commented Feb 9, 2016

Test build #50973 has finished for PR 11134 at commit 1c80127.

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

Contributor

andrewor14 commented Feb 9, 2016

Member

srowen commented Feb 9, 2016

Yes that's fine if it passes tests and it does

Contributor

holdenk commented Feb 9, 2016

LGTM :)

Contributor Author

tedyu commented Feb 10, 2016

@andrewor14
Is there anything I need to do for this ?

Member

srowen commented Feb 10, 2016

@tedyu just have some patience. Sometimes we're just giving others a chance to look before merging. Merged to master

asfgit closed this in

e834e42

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