Skip to content
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.

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

[ADAM-1445] Make apply method for FragmentRDD public. #1446

Merged

Conversation

@fnothaft
Copy link
Member

fnothaft commented Mar 21, 2017

Resolves #1445.

@coveralls
Copy link

coveralls commented Mar 21, 2017

Coverage Status

Coverage decreased (-0.2%) to 80.333% when pulling b461baa on fnothaft:issues/1445-apply-fragmentrdd into 95b2a86 on bigdatagenomics:master.

@heuermh
Copy link
Member

heuermh commented Mar 21, 2017

I want to merge this sooner rather than later as I am about to push a fix for #1439.

@fnothaft
Copy link
Member Author

fnothaft commented Mar 21, 2017

@heuermh SGTM! Fire when ready.

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 21, 2017

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1891/
Test PASSed.

@heuermh heuermh merged commit dbf4f85 into bigdatagenomics:master Mar 21, 2017
2 of 3 checks passed
2 of 3 checks passed
coverage/coveralls Coverage decreased (-0.2%) to 80.333%
Details
codacy/pr Good work! A positive pull request.
Details
default Merged build finished.
Details
@heuermh
Copy link
Member

heuermh commented Mar 21, 2017

Thank you, @fnothaft!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.