[SL-1450] [Bug] Optimizer Does Not Deduplicate Common Metrics

[plypaul]

Is this a new bug in metricflow?

• I believe this is a new bug in metricflow
• I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

If a query includes multiple derived metrics that use a common metric, the optimizer currently does not deduplicate those metrics when combining metric computation branches. If those metrics are not deduplicated, then the resulting SQL will have a column alias error. See: f4691d7#diff-6beb9d2e40a8992c43163d08246fc08088cef8b29bdc6c3ff17b6c173f0f7514R22

Expected Behavior

Common metrics should be deduplicated. See 9fa4aaf#diff-6beb9d2e40a8992c43163d08246fc08088cef8b29bdc6c3ff17b6c173f0f7514L32

Steps To Reproduce

See: f4691d7

Relevant log output

No response

Environment

- metricflow: 0.204.0.dev2

Which database are you using?

No response

Additional Context

No response

_{From SyncLinear.com | SL-1450}