Drop the detach_params from the env

geldata · Feb 20, 2024 · 7c3f953 · 7c3f953
1 parent 29bfc42
commit 7c3f953
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 7 deletions.
diff --git a/edb/pgsql/compiler/__init__.py b/edb/pgsql/compiler/__init__.py
@@ -121,7 +121,6 @@ def compile_ir_to_sql_tree(
             scope_tree_nodes=scope_tree_nodes,
             external_rvars=external_rvars,
             backend_runtime_params=backend_runtime_params,
-            detach_params=detach_params,
         )
 
         ctx = context.CompilerContextLevel(

diff --git a/edb/pgsql/compiler/context.py b/edb/pgsql/compiler/context.py
@@ -495,7 +495,6 @@ class Environment:
     external_rvars: Mapping[Tuple[irast.PathId, str], pgast.PathRangeVar]
     materialized_views: Dict[uuid.UUID, irast.Set]
     backend_runtime_params: pgparams.BackendRuntimeParams
-    detach_params: bool
 
     #: A list of CTEs that implement constraint validation at the
     #: query level.
@@ -518,7 +517,6 @@ def __init__(
             Mapping[Tuple[irast.PathId, str], pgast.PathRangeVar]
         ] = None,
         backend_runtime_params: pgparams.BackendRuntimeParams,
-        detach_params: bool = False,
     ) -> None:
         self.aliases = aliases.AliasGenerator()
         self.output_format = output_format
@@ -536,7 +534,6 @@ def __init__(
         self.materialized_views = {}
         self.check_ctes = []
         self.backend_runtime_params = backend_runtime_params
-        self.detach_params = detach_params
 
 
 # XXX: this context hack is necessary until pathctx is converted

diff --git a/edb/pgsql/compiler/expr.py b/edb/pgsql/compiler/expr.py
@@ -136,9 +136,6 @@ def compile_Parameter(
     else:
         index = ctx.argmap[expr.name].index
         result = pgast.ParamRef(number=index, nullable=not expr.required)
-        if ctx.env.detach_params:
-            # XXX: Do we need this?
-            return result
 
     return pgast.TypeCast(
         arg=result,