InfuseAI / piperider

Code review for data in dbt
https://www.piperider.io/
Apache License 2.0
480 stars 23 forks source link

[Chore] performance tuning #865

Closed qrtt1 closed 1 year ago

qrtt1 commented 1 year ago

PR checklist

What type of PR is this?

enhancement

What this PR does / why we need it:

Make CLI respond quickly

Which issue(s) this PR fixes:

none


Tools

pyinstrument $(command -v piperider)

re-arrange imports

codecov[bot] commented 1 year ago

Codecov Report

Merging #865 (0a0b7d2) into main (9ae345d) will decrease coverage by 1.69%. The diff coverage is 29.91%.

@@            Coverage Diff             @@
##             main     #865      +/-   ##
==========================================
- Coverage   47.82%   46.13%   -1.69%     
==========================================
  Files          71       73       +2     
  Lines        9994    10050      +56     
==========================================
- Hits         4780     4637     -143     
- Misses       5214     5413     +199     
Files Changed Coverage Δ
piperider_cli/cli_utils/compare_with_recipe.py 0.00% <0.00%> (ø)
piperider_cli/cli.py 67.10% <31.81%> (+5.38%) :arrow_up:
piperider_cli/cli_utils/cloud_state.py 60.00% <60.00%> (ø)
piperider_cli/dbtutil.py 75.14% <100.00%> (ø)

... and 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more