Refactor planner filter logic #3253
Labels
refactor
This issue specific to or requires *notable* refactoring of existing codebases and components
Milestone
Comments
nasdf
added
the
refactor
nasdf
added a commit
that referenced
this issue
Dec 5, 2024
## Relevant issue(s) Resolves #3195 ## Description This PR enables aggregate alias targeting in filters. Blocked by #3253 ## Tasks - [x] I made sure the code is well commented, particularly hard-to-understand areas. - [x] I made sure the repository-held documentation is changed accordingly. - [x] I made sure the pull request title adheres to the conventional commit style (the subset used in the project can be found in [tools/configs/chglog/config.yml](tools/configs/chglog/config.yml)). - [x] I made sure to discuss its limitations such as threats to validity, vulnerability to mistake and misuse, robustness to invalidation of assumptions, resource requirements, ... ## How has this been tested? Added and updated integration tests. Specify the platform(s) on which this was tested: - MacOS
|
can i take on this ? |
You are welcome to put together a PR. I wouldn't recommend this as a first time contributor issue though. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Move the planner filtering logic to a
Filternode.The text was updated successfully, but these errors were encountered: