optimize rewrite visitor

[czpmango]

Rewrite expressions in-place.

[codecov-commenter]

Codecov Report

Merging #3053 (35248c2) into master (4dd4a26) will decrease coverage by 0.04%.
The diff coverage is 84.21%.

@@            Coverage Diff             @@
##           master    #3053      +/-   ##
==========================================
- Coverage   85.29%   85.24%   -0.05%     
==========================================
  Files        1289     1289              
  Lines      119969   119978       +9     
==========================================
- Hits       102329   102279      -50     
- Misses      17640    17699      +59     

Impacted Files	Coverage Δ
src/graph/visitor/RewriteVisitor.cpp	76.37% <50.00%> (ø)
src/graph/planner/match/MatchClausePlanner.cpp	98.82% <100.00%> (+0.01%)	⬆️
src/graph/validator/GroupByValidator.cpp	95.28% <100.00%> (ø)
src/graph/validator/MatchValidator.cpp	86.44% <100.00%> (-0.08%)	⬇️
src/meta/processors/parts/GetSpaceProcessor.cpp	60.60% <0.00%> (-18.19%)	⬇️
src/meta/processors/BaseProcessor.h	75.67% <0.00%> (-16.22%)	⬇️
src/graph/context/Result.cpp	85.00% <0.00%> (-9.45%)	⬇️
src/storage/exec/IndexNode.cpp	91.66% <0.00%> (-8.34%)	⬇️
src/graph/executor/StorageAccessExecutor.h	71.79% <0.00%> (-6.42%)	⬇️
src/common/thrift/ThriftClientManager-inl.h	82.97% <0.00%> (-6.39%)	⬇️
... and 29 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f449c17...35248c2. Read the comment docs.