database-mesh / pisanix

A Database Mesh Project Sponsored by SphereEx
https://www.pisanix.io
Apache License 2.0
281 stars 37 forks source link

refactor(runtime): refactor get_avg meta, and refactor handle_avg in executor #394

Closed xuanyuan300 closed 1 year ago

xuanyuan300 commented 1 year ago

Signed-off-by: xuanyuan300 xuanyuan300@gmail.com

What is changed and how it works?

What's Changed: This PR also has changes as follows:

  1. Using lexical parse string to float64 in chunk to fix parse i64 failed.
  2. Added update_columns for RowData trait.
  3. Optimized crc32mod.
codecov[bot] commented 1 year ago

Codecov Report

Merging #394 (bd39600) into master (df61eb4) will increase coverage by 6.48%. The diff coverage is 83.17%.

@@            Coverage Diff             @@
##           master     #394      +/-   ##
==========================================
+ Coverage   66.97%   73.46%   +6.48%     
==========================================
  Files           7        6       -1     
  Lines         427      731     +304     
==========================================
+ Hits          286      537     +251     
- Misses        133      183      +50     
- Partials        8       11       +3     
Flag Coverage Δ
unittests 73.46% <83.17%> (+6.48%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pisa-controller/pkg/proxy/server.go 20.00% <0.00%> (-2.23%) :arrow_down:
pisa-controller/pkg/webhook/server.go 26.66% <0.00%> (ø)
pisa-controller/pkg/proxy/http.go 51.35% <45.90%> (-6.55%) :arrow_down:
pisa-controller/pkg/webhook/injection.go 59.02% <82.56%> (+20.08%) :arrow_up:
pisa-controller/pkg/proxy/builder.go 91.21% <93.57%> (-0.01%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.