thanos-io / promql-engine

Multi-threaded PromQL engine implementation based on the Volcano paper.
Apache License 2.0
131 stars 52 forks source link

Replace function call node #423

Closed fpetkovski closed 3 months ago

fpetkovski commented 3 months ago

This commit replaces the *parser.Call node coming from the AST with a custom logical node type. Long term, we should use our own nodes for everything so that we can enforce certain invariants using our own interface. One method would be having a Children method used for traversal, or having Marshal/Unmarshal methods so we can pass the plan over the wire.