Inferencer removal (#1302)

partiql · Dec 14, 2023 · 8076e6c · 8076e6c · github-actions · Dec 14, 2023
1 parent 6ab6f27
commit 8076e6c
Show file tree

Hide file tree

Showing 9 changed files with 539 additions and 926 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -60,6 +60,8 @@ Thank you to all who have contributed!
 - **Breaking** Removed IR factory in favor of static top-level functions. Change `Ast.foo()`
   to `foo()`
 - **Breaking** Removed `org.partiql.lang.planner.transforms.AstToPlan`. Use `org.partiql.planner.PartiQLPlanner`. 
+- **Breaking** Removed `org.partiql.lang.planner.transforms.PartiQLSchemaInferencer`. In order to achieve the same functionality, one would need to use the `org.partiql.planner.PartiQLPlanner`.
+  - To get the inferred type of the query result, one can do: `(plan.statement as Statement.Query).root.type`
 
 ### Security
 

diff --git a/partiql-lang/src/main/kotlin/org/partiql/annotations/Experimental.kt b/partiql-lang/src/main/kotlin/org/partiql/annotations/Experimental.kt
@@ -20,6 +20,3 @@ annotation class ExperimentalPartiQLCompilerPipeline
 // TODO: Remove from experimental once https://github.com/partiql/partiql-docs/issues/31 is resolved and a RFC is approved
 @RequiresOptIn(message = "Window Function is experimental. It may be changed in the future without notice.", level = RequiresOptIn.Level.ERROR)
 annotation class ExperimentalWindowFunctions
-
-@RequiresOptIn(message = "PartiQLSchemaInferencer is experimental. It may be changed in the future without notice.", level = RequiresOptIn.Level.ERROR)
-annotation class ExperimentalPartiQLSchemaInferencer
diff --git a/partiql-lang/src/main/kotlin/org/partiql/lang/planner/transforms/PartiQLSchemaInferencer.kt b/partiql-lang/src/main/kotlin/org/partiql/lang/planner/transforms/PartiQLSchemaInferencer.kt
diff --git a/partiql-lang/src/test/kotlin/org/partiql/lang/planner/SchemaLoader.kt b/partiql-lang/src/test/kotlin/org/partiql/lang/planner/SchemaLoader.kt