Optimize the oct number representation to with 0o prefix.

[Shylock-Hg]

What changes were proposed in this pull request?

See #1940

Why are the changes needed?

Does this PR introduce any user-facing change?

Yes @Amber1990Zhang

How was this patch tested?

[codecov-io]

Codecov Report

Merging #2014 into master will decrease coverage by 0.18%.
The diff coverage is 94.08%.

@@            Coverage Diff             @@
##           master    #2014      +/-   ##
==========================================
- Coverage   86.90%   86.71%   -0.19%     
==========================================
  Files         636      642       +6     
  Lines       59819    60902    +1083     
==========================================
+ Hits        51984    52811     +827     
- Misses       7835     8091     +256     

Impacted Files	Coverage Δ
src/common/base/Configuration.h	100.00% <ø> (ø)
src/graph/DescribeSpaceExecutor.h	0.00% <ø> (ø)
src/graph/FetchEdgesExecutor.h	0.00% <ø> (ø)
src/graph/FetchExecutor.cpp	87.77% <ø> (-0.53%)	⬇️
src/graph/FetchVerticesExecutor.h	0.00% <ø> (ø)
src/graph/GoExecutor.h	56.25% <ø> (ø)
src/graph/GraphService.h	100.00% <ø> (ø)
src/graph/GroupByExecutor.cpp	85.25% <ø> (-0.06%)	⬇️
src/graph/LimitExecutor.cpp	68.60% <ø> (-0.37%)	⬇️
src/graph/OrderByExecutor.cpp	75.18% <ø> (-0.19%)	⬇️
... and 145 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 7d838d4...938bbc1. Read the comment docs.

[dangleptr]

The behavior has been changed inside this pr.
Do we still want it in 1.x? @whitewum @jude-zhu

[jude-zhu]

I think indicating an octal literal by prepending it with 0 is fine. BTW. 0 is still employed to indicate octal literal in Nebula 2.0.

FYI. same with neo4j

[dangleptr]

I think indicating an octal literal by prepending it with 0 is fine. BTW. 0 is still employed to indicate octal literal in Nebula 2.0.

FYI. same with neo4j

Fine. Please take care of the issue related. Let me close this pr. Thanks

[Shylock-Hg]

I think indicating an octal literal by prepending it with 0 is fine. BTW. 0 is still employed to indicate octal literal in Nebula 2.0.

FYI. same with neo4j

Reservations