restore clause guarding pruneCache check

removing in bigdatagenomics#684 was a mistake, assumed it was redundant but it's not!
ryan-williams · Jun 3, 2015 · ee32831 · ee32831
1 parent 1adc0db
commit ee32831
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/adam-core/src/main/scala/org/bdgenomics/adam/rdd/ShuffleRegionJoin.scala b/adam-core/src/main/scala/org/bdgenomics/adam/rdd/ShuffleRegionJoin.scala
@@ -255,7 +255,7 @@ private case class SortedIntervalPartitionJoin[T, U](binRegion: ReferenceRegion,
         advanceCache(nextLeftRegion.end)
       }
       // ...and whether I need to prune the cache
-      if (prevLeftRegion == null) {
+      if (prevLeftRegion == null || nextLeftRegion.start > prevLeftRegion.start) {
         pruneCache(nextLeftRegion.start)
       }
       // at this point, we effectively do a cross-product and filter; this could probably