-
Notifications
You must be signed in to change notification settings - Fork 308
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-375] Upgrade to Hadoop-BAM 7.0.0. #376
Conversation
0b4dec3
to
a896c26
Compare
Rebased on master. |
a896c26
to
c6fe029
Compare
Rebased. |
Can you rebase one more time, @fnothaft? |
c6fe029
to
d46d777
Compare
Rebased! |
d46d777
to
8722787
Compare
Rererebased. |
8722787
to
963389b
Compare
Rerererebased. |
@@ -49,7 +49,7 @@ | |||
* format. | |||
* | |||
* This reader is based on the FastqInputFormat that's part of Hadoop-BAM, | |||
* found at http://sourceforge.net/p/hadoop-bam/code/ci/master/tree/src/fi/tkk/ics/hadoop/bam/FastqInputFormat.java | |||
* found at http://sourceforge.net/p/hadoop-bam/code/ci/master/tree/src/org.seqdoop.hadoop_bam/FastqInputFormat.java |
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.
This URL returns a 404. Search-replace issue?
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.
Ja. Will fix...
Once the URL is fixed, I'm ready to merge this. Thanks, Frank! |
963389b
to
0c0860b
Compare
Rererererebased + URL fixed. |
[ADAM-375] Upgrade to Hadoop-BAM 7.0.0.
Thanks, Frank! |
Closes #375:
HT to @AndreSchumacher