Feature Request / Improvement
Follow-up to #16626.
IcebergSparkSqlExtensionsParser.parsePlanWithParameters (Spark 4.1) intentionally does not propagate the ParameterContext to the Iceberg parse path, because Iceberg DDL grammars do not accept parameter markers (? / :name).
Today a non-empty parameterContext reaching an Iceberg command is silently ignored. Since that situation would indicate a caller bug rather than a valid query, it would be safer to fail fast (e.g. throw a clear exception) when a non-empty parameterContext is dispatched to an Iceberg command, instead of dropping it silently.
This was raised as a non-blocking review note on #16626 and split out so that PR stays scoped to the binding fix.
Query engine
Spark
Willingness to contribute
Feature Request / Improvement
Follow-up to #16626.
IcebergSparkSqlExtensionsParser.parsePlanWithParameters(Spark 4.1) intentionally does not propagate theParameterContextto the Iceberg parse path, because Iceberg DDL grammars do not accept parameter markers (?/:name).Today a non-empty
parameterContextreaching an Iceberg command is silently ignored. Since that situation would indicate a caller bug rather than a valid query, it would be safer to fail fast (e.g. throw a clear exception) when a non-emptyparameterContextis dispatched to an Iceberg command, instead of dropping it silently.This was raised as a non-blocking review note on #16626 and split out so that PR stays scoped to the binding fix.
Query engine
Spark
Willingness to contribute