-
Notifications
You must be signed in to change notification settings - Fork 206
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
feat: introduce proxy module #732
Conversation
* impl route service with proxy * impl write service with proxy * remove forward module in proxy * refactor code * add tests in write
* refactor: refactor proxy module * cargo fmt * refactor by CR
* feat: impl prom query with proxy * refactor code
* feat: impl stream query with proxy * refactor by CR
Codecov Report
@@ Coverage Diff @@
## main #732 +/- ##
==========================================
- Coverage 68.49% 68.08% -0.41%
==========================================
Files 292 295 +3
Lines 45413 45628 +215
==========================================
- Hits 31104 31066 -38
- Misses 14309 14562 +253
... and 4 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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
* feat: introduce proxy module (apache#715) * impl route service with proxy * impl write service with proxy * remove forward module in proxy * refactor code * add tests in write * feat: impl query with proxy (apache#717) * refactor: refactor proxy module (apache#726) * refactor: refactor proxy module * cargo fmt * refactor by CR * Feat proxy prom query (apache#727) * feat: impl prom query with proxy * refactor code * feat: impl stream write with proxy (apache#737) * feat: impl stream query with proxy (apache#742) * feat: impl stream query with proxy * refactor by CR * feat: introduce proxy module * refactor code * add header in storage service * feat: impl storage service with proxy * make CI happy * refactor code * refactor code * refactor by CR * refactor by CR
Which issue does this PR close?
Closes #
Rationale for this change
Introduce proxy module. Refer to #681 .
What changes are included in this PR?
Are there any user-facing changes?
No.
How does this change test
Integration test.