-
Notifications
You must be signed in to change notification settings - Fork 588
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
Hadoop-BAM and htsjdk upgrades, Spark tool changes. #1469
Conversation
cf52d7f
to
eac87e6
Compare
Ping me once this is ready |
a9debad
to
d5957dd
Compare
d5957dd
to
fcd1fa6
Compare
@droazen Hadoop-BAM has been released now so this is ready for review now whenever you can get to it. There are two other changes included that are unrelated to the tickets listed above:
|
fcd1fa6
to
db98950
Compare
initializeReference(); | ||
initializeReads(sparkContext); // reference must be intialized before reads |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
intialized
-> initialized
Review complete -- back to @cmnbroad |
8ca8f39
to
3fcc2aa
Compare
I think I addressed everything. Back to @droazen. |
👍 merge when ready |
3fcc2aa
to
a2b55b4
Compare
Hadoop-BAM and htsjdk upgrades, Spark tool changes.
Requires upgrades to Hadoop-BAM (not yet released) and htsjdk.
Fixes #1346, #1261, #1175, #1326, #1259, and much of the underlying code for #1270, which will be enabled in a separate PR.