Adds parsing and modeling for INSERT INTO <table name>

CHANGELOG.md

@@ -23,6 +23,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 Thank you to all who have contributed!
 -->
 
+## [0.14.10]
+
+### Experimental Changes
+- **BREAKING**: For the _experimental_ `org.partiql.lang.domains`, the DML targets now use the newly created `TableName`,
+and `IdentifierChain`. With this, we have added parsing support for qualified identifiers in certain DML operations,
+closing [#1595](https://github.com/partiql/partiql-lang-kotlin/issues/1595).
+
 ## [0.14.9]
 
 ### Changed
@@ -1127,7 +1134,8 @@ breaking changes if migrating from v0.9.2. The breaking changes accidentally int
 ### Added
 Initial alpha release of PartiQL.
 
-[Unreleased]: https://github.com/partiql/partiql-lang-kotlin/compare/v0.14.9...HEAD
+[Unreleased]: https://github.com/partiql/partiql-lang-kotlin/compare/v0.14.10...HEAD
+[0.14.10]: https://github.com/partiql/partiql-lang-kotlin/compare/v0.14.9...v0.14.10
 [0.14.9]: https://github.com/partiql/partiql-lang-kotlin/compare/v0.14.8...v0.14.9
 [0.14.8]: https://github.com/partiql/partiql-lang-kotlin/compare/v0.14.7...v0.14.8
 [0.14.7]: https://github.com/partiql/partiql-lang-kotlin/compare/v0.14.6...v0.14.7

README.md

@@ -31,7 +31,7 @@ This project is published to [Maven Central](https://search.maven.org/artifact/o
 
 | Group ID      | Artifact ID           | Recommended Version |
 |---------------|-----------------------|---------------------|
-| `org.partiql` | `partiql-lang-kotlin` | `0.14.9`            |
+| `org.partiql` | `partiql-lang-kotlin` | `0.14.10`           |
 
 
 For Maven builds, add the following to your `pom.xml`:

examples/src/main/java/org/partiql/examples/PartiQLCompilerPipelineAsyncJavaExample.java

@@ -3,6 +3,7 @@
 import com.amazon.ion.IonSystem;
 import com.amazon.ion.system.IonSystemBuilder;
 
+import java.util.List;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.ExecutionException;
 import kotlin.OptIn;
@@ -16,6 +17,8 @@
 import org.partiql.lang.compiler.PartiQLCompilerAsync;
 import org.partiql.lang.compiler.PartiQLCompilerAsyncBuilder;
 import org.partiql.lang.compiler.PartiQLCompilerPipelineAsync;
+import org.partiql.lang.eval.BindingId;
+import org.partiql.lang.eval.BindingName;
 import org.partiql.lang.eval.Bindings;
 import org.partiql.lang.eval.EvaluationSession;
 import org.partiql.lang.eval.ExprValue;
@@ -63,15 +66,30 @@ public void run() {
                 .globals(globalVariables)
                 .build();
 
-        final GlobalVariableResolver globalVariableResolver = bindingName -> {
-            ExprValue value = session.getGlobals().get(bindingName);
-
-            if (value != null) {
-                return new GlobalResolutionResult.GlobalVariable(bindingName.getName());
-            }
-            else {
+        final GlobalVariableResolver globalVariableResolver = new GlobalVariableResolver() {
+            @NotNull
+            @Override
+            public GlobalResolutionResult resolveGlobal(@NotNull BindingId bindingId) {
+                // In this example, we don't allow for qualified identifiers.
+                List<BindingName> parts = bindingId.getParts();
+                if (parts.size() == 1) {
+                    return resolveGlobal(parts.get(0));
+                }
                 return GlobalResolutionResult.Undefined.INSTANCE;
             }
+
+            @NotNull
+            @Override
+            public GlobalResolutionResult resolveGlobal(@NotNull BindingName bindingName) {
+                ExprValue value = session.getGlobals().get(bindingName);
+
+                if (value != null) {
+                    return new GlobalResolutionResult.GlobalVariable(bindingName.getName());
+                }
+                else {
+                    return GlobalResolutionResult.Undefined.INSTANCE;
+                }
+            }
         };
 
         final EvaluatorOptions evaluatorOptions = new EvaluatorOptions.Builder()

gradle.properties

@@ -1,5 +1,5 @@
 group=org.partiql
-version=0.14.9
+version=0.14.10
 
 ossrhUsername=EMPTY
 ossrhPassword=EMPTY

partiql-ast/src/main/kotlin/org/partiql/ast/helpers/ToLegacyAst.kt

@@ -992,14 +992,31 @@ private class AstTranslator(val metas: Map<String, MetaContainer>) : AstBaseVisi
         super.visitStatementDML(node, ctx) as PartiqlAst.Statement
 
     override fun visitStatementDMLInsert(node: Statement.DML.Insert, ctx: Ctx) = translate(node) { metas ->
-        val target = visitIdentifier(node.target, ctx)
+        val target = tableName(node.target)
         val asAlias = node.asAlias?.symbol
         val values = visitExpr(node.values, ctx)
         val conflictAction = node.onConflict?.let { visitOnConflictAction(it.action, ctx) }
         val op = insert(target, asAlias, values, conflictAction)
         dml(dmlOpList(op), null, null, null, metas)
     }
 
+    private fun tableName(id: Identifier): PartiqlAst.TableName = translate(id) { metas ->
+        val identifierChain = when (id) {
+            is Identifier.Symbol -> identifierChain(listOf(identifier(id)))
+            is Identifier.Qualified -> {
+                val root = identifier(id.root)
+                val steps = id.steps.map { identifier(it) }
+                val parts = listOf(root) + steps
+                identifierChain(parts)
+            }
+        }
+        tableName(identifierChain)
+    }
+
+    private fun identifier(id: Identifier.Symbol): PartiqlAst.Identifier = translate(id) { metas ->
+        identifier(id.symbol, id.caseSensitivity.toLegacyCaseSensitivity())
+    }
+
     override fun visitStatementDMLInsertLegacy(
         node: Statement.DML.InsertLegacy,
         ctx: Ctx,
@@ -1016,7 +1033,7 @@ private class AstTranslator(val metas: Map<String, MetaContainer>) : AstBaseVisi
     }
 
     override fun visitStatementDMLUpsert(node: Statement.DML.Upsert, ctx: Ctx) = translate(node) { metas ->
-        val target = visitIdentifier(node.target, ctx)
+        val target = tableName(node.target)
         val asAlias = node.asAlias?.symbol
         val values = visitExpr(node.values, ctx)
         val conflictAction = doUpdate(excluded())
@@ -1026,7 +1043,7 @@ private class AstTranslator(val metas: Map<String, MetaContainer>) : AstBaseVisi
     }
 
     override fun visitStatementDMLReplace(node: Statement.DML.Replace, ctx: Ctx) = translate(node) { metas ->
-        val target = visitIdentifier(node.target, ctx)
+        val target = tableName(node.target)
         val asAlias = node.asAlias?.symbol
         val values = visitExpr(node.values, ctx)
         val conflictAction = doReplace(excluded())
@@ -1122,7 +1139,7 @@ private class AstTranslator(val metas: Map<String, MetaContainer>) : AstBaseVisi
         node: Statement.DML.BatchLegacy.Op.Insert,
         ctx: Ctx,
     ) = translate(node) { metas ->
-        val target = visitIdentifier(node.target, ctx)
+        val target = tableName(node.target)
         val asAlias = node.asAlias?.symbol
         val values = visitExpr(node.values, ctx)
         val conflictAction = node.onConflict?.let { visitOnConflictAction(it.action, ctx) }

partiql-ast/src/main/pig/partiql.ion

@@ -446,7 +446,7 @@ may then be further optimized by selecting better implementations of each operat
         (sum dml_op
             // See the following RFC for more details:
             // https://github.com/partiql/partiql-docs/blob/main/RFCs/0011-partiql-insert.md
-            (insert target::expr as_alias::(? symbol) values::expr conflict_action::(? conflict_action))
+            (insert target::table_name as_alias::(? symbol) values::expr conflict_action::(? conflict_action))
 
             // `INSERT INTO <expr> VALUE <expr> [AT <expr>]` [ON CONFLICT WHERE <expr> DO NOTHING]`
             (insert_value target::expr value::expr index::(? expr) on_conflict::(? on_conflict))
@@ -524,8 +524,13 @@ may then be further optimized by selecting better implementations of each operat
             (all_old)
         )
 
+        // This refers to the <table name> EBNF rule in SQL:1999.
+        (record table_name (id identifier_chain))
+
         // `identifier` can be used for names that need to be looked up with a notion of case-sensitivity.
 
+        // This is a generalization of a qualified identifier in the SQL EBNF that allows for namespaces beyond the
+        // typical (optional) catalog-schema namespace.
         // For both `create_index` and `create_table`, there is no notion of case-sensitivity
         // for table identifiers since they are *defining* new identifiers.  However, for `drop_index` and
         // `drop_table` *do* have the notion of case sensitivity since they are referring to existing names.
@@ -534,6 +539,11 @@ may then be further optimized by selecting better implementations of each operat
         // an element of a type.  (Even though in the Kotlin code each varaint is its own type.)  Hence, we
         // define an `identifier` type above which can be used without opening up an element's domain to all of
         // `expr`.
+        // For how this maps to syntax, consider `INSERT INTO cat1.schema1.table1`. In this scenario, `table1` is the
+        // head. The qualifier is `cat1.schema1`.
+        (record identifier_chain (parts (* identifier 1)))
+
+        // This refers to a regular/delimited identifier in the SQL EBNF.
         (product identifier name::symbol case::case_sensitivity)
 
         // Represents `<expr> = <expr>` in a DML SET operation.  Note that in this case, `=` is representing
@@ -853,7 +863,7 @@ may then be further optimized by selecting better implementations of each operat
 
             // The "target" of a DML operation, i.e. the table targeted for manipulation with INSERT, UPDATE, etc.
             // This is a discrete type so it can be permuted in later domains to affect every use.
-            (record dml_target (identifier identifier))
+            (record dml_target (identifier table_name))
 
             // An assignment within a SET clause.
             (record set_assignment

partiql-lang/src/main/kotlin/org/partiql/lang/eval/BindingId.kt

@@ -0,0 +1,27 @@
+package org.partiql.lang.eval
+
+/**
+ * Represents a namespaced global binding.
+ * @property parts a collection of the parts of the identifier.
+ */
+data class BindingId(
+    val parts: List<BindingName>
+) : Iterable<BindingName> {
+
+    init {
+        assert(parts.any())
+    }
+
+    constructor(vararg names: BindingName) : this(names.toList())
+
+    override fun iterator(): Iterator<BindingName> {
+        return parts.iterator()
+    }
+
+    override fun toString(): String {
+        return when (this.parts.size > 1) {
+            true -> parts.joinToString(".")
+            false -> parts[0].toString()
+        }
+    }
+}

partiql-lang/src/main/kotlin/org/partiql/lang/eval/Bindings.kt

@@ -65,6 +65,13 @@ data class BindingName(val name: String, val bindingCase: BindingCase) {
      * Compares [name] to [otherName] using the rules specified by [bindingCase].
      */
     fun isEquivalentTo(otherName: String?) = otherName != null && name.isBindingNameEquivalent(otherName, bindingCase)
+
+    override fun toString(): String {
+        return when (bindingCase) {
+            BindingCase.SENSITIVE -> "\"$name\""
+            BindingCase.INSENSITIVE -> "$name"
+        }
+    }
 }
 
 /**

partiql-lang/src/main/kotlin/org/partiql/lang/planner/GlobalVariableResolver.kt

@@ -1,6 +1,7 @@
 package org.partiql.lang.planner
 
 import org.partiql.lang.eval.BindingCase
+import org.partiql.lang.eval.BindingId
 import org.partiql.lang.eval.BindingName
 
 /**
@@ -17,6 +18,21 @@ sealed class GlobalResolutionResult {
      */
     data class GlobalVariable(val uniqueId: String) : GlobalResolutionResult()
 
+    /**
+     * A success case. Refers to a variable that is contained within a namespace (AKA catalog/schema).
+     */
+    abstract class NamespacedVariable(
+        val uniqueId: String
+    ) : GlobalResolutionResult() {
+        /**
+         * When attempting to resolve a qualified identifier, say `FROM cat1.schema1.table1.attr1.attr2`, the [GlobalVariableResolver]
+         * _may_ resolve only the first few steps of the identifier, say `cat1.schema1.table1`. Therefore, it must
+         * return the resolved [uniqueId] (say `cat1:::schema1:::table1`) and the remaining steps `attr1.attr2` in order for the planner
+         * to convert these to path expressions.
+         */
+        abstract fun getRemainingSteps(): List<BindingName>
+    }
+
     /** A failure case, indicates that resolution did not match any variable. */
     object Undefined : GlobalResolutionResult()
 }
@@ -56,6 +72,19 @@ fun interface GlobalVariableResolver {
      */
     fun resolveGlobal(bindingName: BindingName): GlobalResolutionResult
 
+    /**
+     * Resolves a potentially qualified [BindingId] in the database environment.
+     * By default, if the [bindingId] does not have a qualified identifier passed to it, it will resolve using just
+     * the unqualified identifier. If it is qualified, it will return a [GlobalResolutionResult.Undefined].
+     */
+    fun resolveGlobal(bindingId: BindingId): GlobalResolutionResult {
+        val parts = bindingId.parts
+        if (parts.size == 1) {
+            return resolveGlobal(parts.last())
+        }
+        return GlobalResolutionResult.Undefined
+    }
+
     companion object {
 
         val EMPTY = GlobalVariableResolver { GlobalResolutionResult.Undefined }

partiql-lang/src/main/kotlin/org/partiql/lang/planner/PlanningProblemDetails.kt

@@ -2,6 +2,9 @@ package org.partiql.lang.planner
 
 import org.partiql.errors.ProblemDetails
 import org.partiql.errors.ProblemSeverity
+import org.partiql.lang.eval.BindingCase
+import org.partiql.lang.eval.BindingId
+import org.partiql.lang.eval.BindingName
 
 /**
  * Contains detailed information about errors that may occur during query planning.
@@ -32,15 +35,29 @@ sealed class PlanningProblemDetails(
             }
         )
 
-    data class UndefinedDmlTarget(val variableName: String, val caseSensitive: Boolean) :
-        PlanningProblemDetails(
-            ProblemSeverity.ERROR,
-            {
-                "Data manipulation target table '$variableName' is undefined. " +
-                    "Hint: this must be a name in the global scope. " +
-                    quotationHint(caseSensitive)
-            }
+    data class UndefinedDmlTarget(val id: BindingId) : PlanningProblemDetails(
+        ProblemSeverity.ERROR,
+        {
+            "Data manipulation target table '$id' is undefined. Hint: this must be a name in the global scope. " + quotationHint(id.parts.last().bindingCase == BindingCase.SENSITIVE)
+        }
+    ) {
+
+        private val lastPart = id.parts.last()
+        val variableName = lastPart.name
+        val caseSensitive = lastPart.bindingCase == BindingCase.SENSITIVE
+
+        constructor(variableName: String, caseSensitive: Boolean) : this(
+            BindingId(
+                BindingName(
+                    variableName,
+                    when (caseSensitive) {
+                        true -> BindingCase.SENSITIVE
+                        false -> BindingCase.INSENSITIVE
+                    }
+                )
+            )
         )
+    }
 
     data class VariablePreviouslyDefined(val variableName: String) :
         PlanningProblemDetails(