FiloDB icon indicating copy to clipboard operation
FiloDB copied to clipboard

Published 20 hours ago • verified publisher icon filodb

fix(query): override RemoteExec::doExecute instead of execute()

Open alextheimer opened this issue 2 years ago • 1 comments

Pull Request checklist

  • [x] The commit(s) message(s) follows the contribution guidelines ?
  • [x] Tests for the changes have been added (for bug fixes / features) ?
  • [ ] Docs have been added / updated (for bug fixes / features) ?

Currently, RemoteExec plans override the usual execute logic; this means that common tasks like RangeVectorTransformer application will be missed if they are not explicitly performed in the overridden execute.

This PR moves the overridden logic into doExecute, so all common execute tasks will be performed automatically.

alextheimer avatar Dec 09 '22 02:12 alextheimer