Skip to content

fix: use Buckets::Any for Values

Arseniy Volynets requested to merge av/fix-values-buckets into main

Summarize the changes

  • fix: use Buckets::Any for Values
  • Earlier we returned empty buckets for Values/ValuesRow nodes and then choosed random bucket for execution if they were a subtree root.
  • Buckets::Any provides the same semantics, use it instead and simplify empty_query_result function, which now always returns empty result.

Ensure that

  • New code is covered by unit and integration tests.
  • Related issues would be automatically closed with gitlab's closing pattern (Closes #issue_number).
  • Public modules are documented (check the rendered version with cargo doc --open).
  • (if PEST grammar has changed) EBNF grammar reflects these changes (check the result with railroad diagram generator.

Next steps

Edited by Arseniy Volynets

Merge request reports