j1wonpark opened a new issue, #16683: URL: https://github.com/apache/iceberg/issues/16683
### 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 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
