Some changes to avoid confusion

[wenhaocs]

The original naming in the storage plan tree execution is confusing: when adding dependents to a node, all its children are set to hasDependents_. I believe isRoot is more accurate.

[codecov-commenter]

Codecov Report

Merging #3339 (3f1ef54) into master (fe68b0e) will decrease coverage by 0.04%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #3339      +/-   ##
==========================================
- Coverage   85.27%   85.22%   -0.05%     
==========================================
  Files        1276     1276              
  Lines      118999   119004       +5     
==========================================
- Hits       101475   101425      -50     
- Misses      17524    17579      +55     

Impacted Files	Coverage Δ
src/common/expression/ContainerExpression.cpp	90.00% <66.66%> (-2.13%)	⬇️
src/common/utils/MetaKeyUtils.cpp	81.31% <100.00%> (ø)
src/graph/validator/test/YieldValidatorTest.cpp	91.78% <100.00%> (ø)
src/parser/test/ParserTest.cpp	100.00% <100.00%> (ø)
src/storage/exec/RelNode.h	79.06% <100.00%> (ø)
src/storage/exec/StoragePlan.h	100.00% <100.00%> (ø)
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/common/thrift/ThriftClientManager-inl.h	82.97% <0.00%> (-6.39%)	⬇️
... and 27 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 8872973...3f1ef54. Read the comment docs.

[panda-sheep]

Good!

[yixinglu]

LGTM

[SuperYoko]

LGTM