Add ability to perform a "range join" to RDDs of ADAMRecords. about adam HOT 5 CLOSED

I want to use the AdamContext.adamLoadVariantContext method in the depth calculation, rather than parsing our own VCFs by hand, but I'm going to have to fix this issue first: #127

from adam.

You may want to hold on that fix; I'd say that issue #127 should be a "won't fix" as the new VCF code from Mt. Sinai should address this issue.

from adam.

@nealsid is this related to the changes you were talking about, on IRC?

from adam.

One aspect where our pending PR will help is to isolate the fields that need to be compared for the join into ADAMVariant. So you can write utility functions to do field-by-field comparison (which already exists), or the necessary for logic to do range comparisons on ADAMVariant. Then a new record type that needs to be attached to a location on the chromosome can just contain ADAMVariant and get the range join almost for free, rather than having to extend it to look at the location fields directly in a new record type.
But after writing this out of course I realized that joining based on genomics coordinates may not only be related to variants and annotation databases, so maybe we should factor it out a bit more even.

from adam.

This has been fixed for a while; closing.

from adam.