Walaa Eldin Moustafa

@phd3, where would these expressions be stored in the case of the Hive Metastore connector?

> But I've requirement of converting pure (LogicalPlan -> String) to re-generate sql. Could you please clarify what kind of logical plan representation this refers to? If you are referring...

It is work in progress. Please check out the [trino2rel](https://github.com/linkedin/coral/tree/master/coral-trino/src/main/java/com/linkedin/coral/trino/trino2rel) package.

I am not clear on the motivation. So obviously, current test cases do not have this issue. Did you run to them in production views (that they use backquotes)? If...

By any chance are we using the wrong dialect to generate this SQL? Dialects allow customizing the quotes if that was the issue. See https://github.com/linkedin/coral/blob/master/coral-trino/src/main/java/com/linkedin/coral/trino/rel2trino/TrinoSqlDialect.java#L18.

@amogh-jahagirdar very good points. Following this line of thought, we should probably also address the question of whether the SQL text should be validated in Iceberg/the API implementation, or validation...

Should the spec define rules for schema evolution of views across different versions? The implementation should probably enforce them.

> * Interface name `ViewVersion` not ideal, but no good alternative. How about `ViewMetadataVersion`, `ViewMetadataSnapshot`, or simply `MetadataSnapshot`? I am still unclear on whether we could store different dialects for...

> > Multiple dialects are available at the same time, so in one version. > What is an API example to achieve that? > Each `Version` contains a list of...

> Not exposed currently. Do you have a use case? What do u need in the API to achieve it? I do not necessarily have a use case, but was...