-
Notifications
You must be signed in to change notification settings - Fork 5.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
*: move hint and block hint processor to util directory #15897
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #15897 +/- ##
================================================
- Coverage 80.5573% 80.4090% -0.1484%
================================================
Files 505 505
Lines 135347 135006 -341
================================================
- Hits 109032 108557 -475
- Misses 17836 17940 +104
- Partials 8479 8509 +30 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/run-all-tests |
cherry pick to release-4.0 in PR #15900 |
What problem does this PR solve?
Problem Summary:
Not a problem, just move code around. Split from #15895.
What is changed and how it works?
What's Changed:
Move code related to
hintProcessor
andBlockHintProcessor
toutil/hint/
directory, because I want to use them in #15895, otherwise, I would have import cycle.How it Works:
Related changes
Check List
Tests
Side effects
N/A
Release note