Sharded bam writer merge

[iliat]

No description provided.

[googlebot]

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

[googlebot]

CLAs look good, thanks!

[deflaux]

extremely minor nit: lots of whitespace changes in here that are not needed

[iliat]

@deflaux @davidadamsphd
Sorry for the late reply - I'm back now, will address the comments and try to get to merge-able state this week.

[iliat]

@deflaux I addressed a bunch of comments and added an integration test. PTAL.
I will file issues for subsequent refactoring and improvements.

[deflaux]

@iliat thanks for those fixes. Please go ahead and file those issues?

[iliat]

Already did :)
#141

On Wed, Sep 9, 2015 at 3:08 PM, Nicole Deflaux notifications@github.com
wrote:

@iliat https://github.com/iliat thanks for those fixes. Please go ahead
and file those issues?

—
Reply to this email directly or view it on GitHub
#140 (comment)
.

[deflaux]

that's great, thanks!

[iliat]

So @deflaux , ok to merge ?

[deflaux]

Its okay by me! Not sure if @davidadamsphd had any feedback.

[davidadamsphd]

Overall looks good. No real feedback.