Simplify predicate pushdown state tracking

[tlento]

The original implementation of the state tracker for predicate pushdown
was built around an assumption that node-level and node-path access would
be useful for propagating the application of filter predicates back
up the dependency chain.

After working towards an implementation of that logic it's clear that this
is not particularly helpful. Although this tracking is theoretically useful
for debugging scenarios the node-visit-level logging appears adequate, so we
simplify our tracking before adding additional logic.

[github-actions]

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]

• Enable PredicatePushdownOptimization for all MetricFlowEngine queries #1308
• Enforce order of application for DataflowPlanOptimizers #1307
• Enable SourceScanOptimizer in query rendering tests #1306
• Add SQL rendering tests for pushdown edge cases #1305
• Remove duplicated WhereConstraintNodes in predicate pushdown #1304
• Add direct tests for predicate pushdown optimizer #1303
• Track and propagate applied where filter specs to outer plan nodes #1302
• Simplify predicate pushdown state tracking #1301 👈
• Consolidate where constraint predicate pushdown management #1300
• Move categorical dimension filter pushdown to PredicatePushdownOptimizer #1286
• Move node dataset resolver subquery ids into a separate namespace #1280
• Verify column sets match in optimizer pushdown evaluation #1279
• Update API for requesting dataflow plan optimization #1278
• Store a list of input WhereFilterSpecs in the WhereConstraintNode #1277
• Enable DataflowPlanOptimizers for query rendering tests #1263
• Add PredicatePushdownOptimizer in tracking-only mode #1262
• Add integration tests for filters against various join types #1240
• Enable predicate pushdown for categorical dimension filters #1227
• Add semantic_model_origin property to LinkableElement interface #1230
• Add element type property to LinkableElement interface #1226
• Encapsulate time range constraint application in more general method #1225
• Rename PredicatePushdownParameters to PredicatePushdownState #1224
• Make source_semantic_models property accessible from a DataflowPlanNode #1218
• Use pushdown params to disable time range constraint pushdown #1216 : 1 other dependent PR (#1217 )
• Encapsulate time range constraints for predicate pushdown #1215
• Add conversion metric rendering tests #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

[courtneyholcomb]

🧹 🧹

[tlento]

Merge activity

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