Add Default Dialect at Dataset Level #16
Replies: 3 comments 3 replies
-
|
I would say that a default SQL dialect may be required at an even higher layer: the whole model. Joining between a dataset sourced in Snowflake vs. a dataset sourced in Databricks doesn't really make much sense, and any consumer of an OSI needs to know what host SQL language to write |
Beta Was this translation helpful? Give feedback.
-
|
Yeah. I think this comes down to how we think people are going to use these. In a world where there is one semantic model that is deployed to several providers, then having different dialects at the field could make sense. However, if you are going to have one model per provider, and some form of composition to re-use the parts that are common across all, then having the dialect at the top level makes sense. |
Beta Was this translation helpful? Give feedback.
-
|
dmzolotarev and cpimhoff Agreed on having a default dialect field at the model level. Does one of you want to create a PR with this change ? |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Currently, every field must specify its dialect.
But can it possibly be that in one dataset fields will be from a different dialects?
I'm struggling to come up with such usecases, but it any way, we can move
dialectto a dataset level, with an option to override it on field level. Fields will inherit dataset-level dialect unless they specify their own dialectThis will reduce verbosity.
Before (verbose):
After (concise):
Beta Was this translation helpful? Give feedback.
All reactions