IMO, it would be preferable to have:

```
def adamBQSR(knownSnps: SnpTable): RDD[ADAMRecord] = {
     BaseQualityRecalibration(rdd, rdd.context.broadcast(knownSnps))
```

I think this is a bit cleaner, would be glad to hear comments.