cube icon indicating copy to clipboard operation
cube copied to clipboard

refactor(cubesql): Use Async lock for Server's sessions

Open ovr opened this issue 1 year ago • 1 comments

Hello!

This PR doesn't provide new functionallity or bug fix.

Thanks

ovr avatar Aug 01 '22 13:08 ovr

Codecov Report

Merging #5040 (300bb83) into master (a3a8b89) will increase coverage by 22.97%. The diff coverage is 80.00%.

@@             Coverage Diff             @@
##           master    #5040       +/-   ##
===========================================
+ Coverage   60.02%   82.99%   +22.97%     
===========================================
  Files         139      129       -10     
  Lines       11593    19616     +8023     
  Branches     2918        0     -2918     
===========================================
+ Hits         6959    16281     +9322     
+ Misses       4130     3335      -795     
+ Partials      504        0      -504     
Flag Coverage Δ
cube-backend ?
cubesql 82.99% <80.00%> (?)

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

Impacted Files Coverage Δ
...ne/information_schema/postgres/pg_stat_activity.rs 0.00% <0.00%> (ø)
rust/cubesql/cubesql/src/sql/postgres/shim.rs 65.80% <0.00%> (ø)
rust/cubesql/cubesql/src/sql/mysql/service.rs 52.00% <76.92%> (ø)
rust/cubesql/cubesql/src/sql/session_manager.rs 87.50% <81.81%> (ø)
rust/cubesql/cubesql/src/sql/postgres/service.rs 70.21% <82.35%> (ø)
rust/cubesql/cubesql/src/compile/mod.rs 81.10% <90.90%> (ø)
...ile/engine/information_schema/mysql/processlist.rs 89.83% <100.00%> (ø)
packages/cubejs-backend-shared/src/errors.ts
...-query-orchestrator/src/orchestrator/QueryCache.ts
...ckages/cubejs-server-core/src/core/agentCollect.ts
... and 265 more

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

codecov[bot] avatar Aug 01 '22 14:08 codecov[bot]