fix: wrong hash calculation of plan subtree
Summarize the changes
- fix: wrong hash calculation of plan subtree
- We didn't traverse output during subtree traversal when calculating hash. Some nodes (Motion, Projection) store non-trivial information, which allows to distinguish different plans
- fix this by traversing subtree fully
Ensure that
-
New code is covered by unit and integration tests. -
Related issues would be automatically closed with gitlab's closing pattern (Closes #1, #2
). -
Public modules are documented (check the rendered version withcargo doc --open
). -
(if PEST grammar is changed) EBNF grammar reflects these changes (check the result with railroad diagram generator.
Next steps
- Cherry-pick to: none
- Update sbroad submodule in picodata/picodata.
- (if EBNF grammar is changed) create a follow-up issue in picodata/docs.
Edited by Denis Smirnov