talatuyarer commented on issue #15296: URL: https://github.com/apache/iceberg/issues/15296#issuecomment-5460562665
@pvary Thanks. This should not be blocker to have View support in Flink. The exact scenario can't happen from Flink. Flink has no CREATE OR REPLACE VIEW dispatch path at all, so there's no Flink path that replaces a view wholesale via CREATE. #17874's ALTER VIEW ... AS goes through `replaceVersion()` carrying only the flink representation — and core's `replace.drop-dialect.allowed` guard (default false) rejects the replace when it would drop another engine's dialect. That's `testAlterViewAsDroppingOtherDialectFails`, and it's called out in the #17874 description and i will put it on docs too. So from Flink there is no silent deletion — deletion only happens if the user explicitly opts in with `replace.drop-dialect.allowed=true`. -- 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]
