Probably an issue for another time, but right now their implicit assumption that the dbsnp sites file and adam records have same contig names, i.e. one can't be stored as 'chr1' while the other is stored under '1'.  Perhaps this obvious to end user, but we could also put another check here.