Skip to content

fix: wrong hash calculation of plan subtree

Arseniy Volynets requested to merge av/fix-hash into main

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 with cargo doc --open).
  • (if PEST grammar is changed) EBNF grammar reflects these changes (check the result with railroad diagram generator.

Next steps

Edited by Denis Smirnov

Merge request reports