Enforce order of application for DataflowPlanOptimizers

dbt-labs / metricflow

MetricFlow allows you to define, build, and maintain metrics in code.

https://docs.getdbt.com/docs/build/about-metricflow

Other

1.12k stars 92 forks source link

Enforce order of application for DataflowPlanOptimizers #1307

Closed tlento closed 2 months ago

tlento commented 2 months ago

We want to make sure optimizers are run in the same order from all outside callers, and a subsequent change will broaden our exposure to dbt cloud service callers. Rather than expecting everybody to pick the "right" optimization order, we simply accept a set of optimizations to run and order them internally.

github-actions[bot] commented 2 months ago

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

tlento commented 2 months ago

#1308
#1307 👈
#1306
#1305
#1304
#1303
#1302
#1301
#1300
#1286
#1280
#1279
#1278
#1277
#1263
#1262
#1240
#1227
#1230
#1226
#1225
#1224
#1218
#1216 : 1 other dependent PR (#1217 )
#1215
#1214
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tlento and the rest of your teammates on Graphite

tlento commented 2 months ago

Merge activity

Jun 25, 5:24 PM PDT: @tlento started a stack merge that includes this pull request via Graphite.
Jun 25, 5:54 PM PDT: Graphite rebased this pull request as part of a merge.
Jun 25, 5:58 PM PDT: @tlento merged this pull request with Graphite.