Skip to content

Commit

Permalink
Revert "More debugging instrumentation for Tim Poterba (#239)"
Browse files Browse the repository at this point in the history
This reverts commit eb2cbba.
  • Loading branch information
lgruen committed Nov 22, 2022
1 parent 7cbfd74 commit 06d31eb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
9 changes: 1 addition & 8 deletions hail/src/main/scala/is/hail/expr/ir/TableIR.scala
Original file line number Diff line number Diff line change
Expand Up @@ -1060,14 +1060,7 @@ case class PartitionZippedIndexedNativeReader(specLeft: AbstractTypedCodecSpec,

cb.ifx(endIndex > startIndex, {
val leafNode = indexResult.loadField(cb, 2)
.handle(cb, {
cb._fatal("leaf node missing!",
"\n context: ", cb.strValue(ctxStruct),
"\n query result: ", cb.strValue(indexResult),
"\n first leaf: ", index.queryIndex(cb, outerRegion, 0L),
"\n last leaf: ", index.queryLastIndex(cb, outerRegion)
)
})
.get(cb)
.asBaseStruct

val leftSeekAddr = leftOffsetFieldIndex match {
Expand Down
3 changes: 1 addition & 2 deletions hail/src/main/scala/is/hail/io/index/StagedIndexReader.scala
Original file line number Diff line number Diff line change
Expand Up @@ -156,8 +156,7 @@ class StagedIndexReader(emb: EmitMethodBuilder[_], spec: AbstractIndexSpec) {
}, region, partitionBoundLeftEndpoint, leansRight)
}

def queryLastIndex(cb: EmitCodeBuilder, region: Value[Region]): SBaseStructValue = {queryIndex(cb, region, cb.memoize(cb.memoize(metadata.invoke[Long]("nKeys"))-1))}
def queryIndex(cb: EmitCodeBuilder, region: Value[Region], absIndex: Value[Long]): SBaseStructValue = {
private def queryIndex(cb: EmitCodeBuilder, region: Value[Region], absIndex: Value[Long]): SBaseStructValue = {
cb.invokeSCode(
cb.emb.ecb.getOrGenEmitMethod("queryIndex",
("queryIndex", this),
Expand Down

0 comments on commit 06d31eb

Please sign in to comment.