Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Enhance(dashboard): optimize get policies request #1539

Merged
merged 1 commit into from
Dec 16, 2022

Conversation

topjohncian
Copy link
Member

Execute a single SQL instead of a batch of slow searches.

@codecov
Copy link

codecov bot commented Nov 24, 2022

Codecov Report

Merging #1539 (e1c6e3f) into master (4519dc0) will decrease coverage by 0.82%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1539      +/-   ##
==========================================
- Coverage   90.05%   89.23%   -0.83%     
==========================================
  Files          99       99              
  Lines        6710     8494    +1784     
==========================================
+ Hits         6043     7580    +1537     
- Misses        503      750     +247     
  Partials      164      164              
Impacted Files Coverage Δ
pkg/conf/conf.go 72.72% <0.00%> (-4.55%) ⬇️
pkg/util/io.go 59.25% <0.00%> (-4.38%) ⬇️
models/group.go 78.57% <0.00%> (-4.04%) ⬇️
pkg/hashid/hash.go 80.64% <0.00%> (-3.36%) ⬇️
pkg/task/pool.go 96.77% <0.00%> (-3.23%) ⬇️
pkg/filesystem/driver/remote/handler.go 81.00% <0.00%> (-3.15%) ⬇️
pkg/request/request.go 80.28% <0.00%> (-2.76%) ⬇️
pkg/cluster/pool.go 92.24% <0.00%> (-2.56%) ⬇️
models/migration.go 77.19% <0.00%> (-2.50%) ⬇️
pkg/serializer/setting.go 93.47% <0.00%> (-1.88%) ⬇️
... and 81 more

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

@HFO4 HFO4 merged commit fd59d1b into cloudreve:master Dec 16, 2022
@HFO4
Copy link
Member

HFO4 commented Dec 16, 2022

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants