assert isRealizable in executeLiteral, not execute

hail-is · Oct 8, 2024 · 8fb6e01 · 8fb6e01
1 parent 3c702c0
commit 8fb6e01
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/hail/src/main/scala/is/hail/backend/local/LocalBackend.scala b/hail/src/main/scala/is/hail/backend/local/LocalBackend.scala
@@ -182,7 +182,6 @@ class LocalBackend(val tmpdir: String) extends Backend with BackendWithCodeCache
     ctx.time {
       TypeCheck(ctx, ir)
       Validate(ir)
-      assert(ir.typ.isRealizable)
       val queryID = Backend.nextID()
       log.info(s"starting execution of query $queryID of initial size ${IRSize(ir)}")
       ctx.irMetadata.semhash = SemanticHash(ctx)(ir)
@@ -194,6 +193,7 @@ class LocalBackend(val tmpdir: String) extends Backend with BackendWithCodeCache
   def executeLiteral(irStr: String): Int =
     withExecuteContext { ctx =>
       val ir = IRParser.parse_value_ir(irStr, IRParserEnvironment(ctx, persistedIR.toMap))
+      assert(ir.typ.isRealizable)
       execute(ctx, ir) match {
         case Left(_) => throw new HailException("Can't create literal")
         case Right((pt, addr)) =>

diff --git a/hail/src/main/scala/is/hail/backend/service/ServiceBackend.scala b/hail/src/main/scala/is/hail/backend/service/ServiceBackend.scala
@@ -335,7 +335,6 @@ class ServiceBackend(
     ctx.time {
       TypeCheck(ctx, ir)
       Validate(ir)
-      assert(ir.typ.isRealizable)
       val queryID = Backend.nextID()
       log.info(s"starting execution of query $queryID of initial size ${IRSize(ir)}")
       ctx.irMetadata.semhash = SemanticHash(ctx)(ir)

diff --git a/hail/src/main/scala/is/hail/backend/spark/SparkBackend.scala b/hail/src/main/scala/is/hail/backend/spark/SparkBackend.scala
@@ -542,7 +542,6 @@ class SparkBackend(
     ctx.time {
       TypeCheck(ctx, ir)
       Validate(ir)
-      assert(ir.typ.isRealizable)
       ctx.irMetadata.semhash = SemanticHash(ctx)(ir)
       try {
         val lowerTable = getFlag("lower") != null
@@ -558,6 +557,7 @@ class SparkBackend(
   def executeLiteral(irStr: String): Int =
     withExecuteContext { ctx =>
       val ir = IRParser.parse_value_ir(irStr, IRParserEnvironment(ctx, persistedIR.toMap))
+      assert(ir.typ.isRealizable)
       execute(ctx, ir) match {
         case Left(_) => throw new HailException("Can't create literal")
         case Right((pt, addr)) =>