statistics: rename queue files for consistency
What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tidb/issues/55906
Problem Summary:
What changed and how does it work?
- Renamed
queue_v2_xxx.gotoqueue_xxx.go. Because we only have one version now. - Refactored
FetchAllTablesAndBuildAnalysisJobsas a method of the priority queue.
Check List
Tests
- [x] Unit test
- [ ] Integration test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test
- [ ] I checked and no code files have been changed.
Side effects
- [ ] Performance regression: Consumes more CPU
- [ ] Performance regression: Consumes more Memory
- [ ] Breaking backward compatibility
Documentation
- [ ] Affects user behaviors
- [ ] Contains syntax changes
- [ ] Contains variable changes
- [ ] Contains experimental features
- [ ] Changes MySQL compatibility
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.
None
Codecov Report
Attention: Patch coverage is 71.26866% with 154 lines in your changes missing coverage. Please review.
Project coverage is 56.9039%. Comparing base (
e13bfeb) to head (a0c96f0). Report is 29 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #56732 +/- ##
=================================================
- Coverage 73.3262% 56.9039% -16.4224%
=================================================
Files 1630 1786 +156
Lines 450312 646607 +196295
=================================================
+ Hits 330197 367945 +37748
- Misses 99833 253880 +154047
- Partials 20282 24782 +4500
| Flag | Coverage Δ | |
|---|---|---|
| integration | 38.6749% <4.2910%> (?) |
|
| unit | 72.8942% <71.2686%> (+0.4341%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
| Components | Coverage Δ | |
|---|---|---|
| dumpling | 52.9478% <ø> (ø) |
|
| parser | ∅ <ø> (∅) |
|
| br | 52.5250% <ø> (+6.9362%) |
:arrow_up: |
/retest
/retest
/retest
[LGTM Timeline notifier]
Timeline:
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: AilinKid, elsa0520, lance6716
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~pkg/statistics/OWNERS~~ [AilinKid,elsa0520]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment