Merge tracing fixes to 24-1 #5458
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
serbel324
commented
Jun 11, 2024
serbel324
commented
- Implemented first version of dynamic jaeger tracing configuration (Implemented first version of dynamic jaeger tracing configuration #1484)
- Moved sampler and throttler tests to jaeger_tracing folder (Moved sampler and throttler tests to jaeger_tracing folder #1572)
- Made tvm url optional (Made tvm url optional #1591)
- Remove obsolete controllers (Remove obsolete controllers #1577)
- Removed required fields from tracing configs (Removed required fields from tracing configs #1636)
- Removed aborts in service initialization (Removed aborts in service initialization #1652)
- Fixed several issues in tracing configuration in static.py (Fixed several issues in tracing configuration in static.py #1952)
- Implemented batching in wilson uploader (Implemented batching in wilson uploader #1955)
- Made wilson uploader limits configurable (Made wilson uploader limits configurable #1782)
- Added request type to request discriminator (Added request type to request discriminator #2058)
- Removed GetInternalRequestType from IRequestProxyCtx (Removed GetInternalRequestType from IRequestProxyCtx #2087)
- Wilson parallel upload (Wilson parallel upload #1875)
- Rewritten sampling and throttling controls to share limits and implement scoping (Rewritten sampling and throttling controls to share limits and implement scoping #1902)
- Fixed missing peerdir in grpc_services (Fixed missing peerdir in grpc_services #2132)
- Renamed several tracing parameters (Renamed several tracing parameters #2134)
- Added database attribute to grpc request proxy span (Added database attribute to grpc request proxy span #2161)
- Added database to request discriminator (Added database to request discriminator #2221)
- Implemented tracing for bulk row upsert (Implemented tracing for bulk row upsert #2218)
- Implemented tracing sampling by db (Implemented tracing sampling by db #2220)
- Removed redundant wilson uploader parameter (Removed redundant wilson uploader parameter #2270)
- Tracing scope to tracing rule rename (Tracing scope to tracing rule rename #2268)
- Removed outdated comment (Removed outdated comment #2339)
- Tracing levels proposal implementation (Tracing levels proposal implementation #2415)
- Missing tracing in table/query service (Missing tracing in table/query service #2403)
- Changed tracing levels in PDisk (Changed tracing levels in PDisk #2473)
- clang14 compat (clang14 compat #2137)
- Removed tracing level Full (Removed tracing level Full #2658)
- Added proper request finishing in grpc request proxy (Added proper request finishing in grpc request proxy #2568)
- Discovery request check spans (Discovery request check spans #2641)
- Changed default verbosity for external traces (Changed default verbosity for external traces #2688)
- Implemented selector allowing for several request types (Implemented selector allowing for several request types #2810)
- Codestyle fix (Codestyle fix #2903)
- Sql request type selectors (Sql request type selectors #2902)
- Added tests for jaeger tracing configurator (Added tests for jaeger tracing configurator #3309)
- Fixed double finishing of spans in grpc request proxy (Fixed double finishing of spans in grpc request proxy #3208)
- Changed default behaviour of external_throttling section (Changed default behaviour of external_throttling section #3408)
…b-platform#1484) * Implemented first version of dynamic jaeger tracing configuration * First semi-done version * Successfully conquered configurations * Removed useless output + slight refactoring * Fix of test client
* Made tvm url optional * Python codestyle fix * Changed validation accordingly
* Removed obsolete controls * Fix of json_proxy_proto.h * Reserved deleted field id
* Removed required protobuf fields * Typo fix * Removed max_burst from required fields * Reverted changed to repeated fields
|
⚪
|
|
⚪
|
|
⚪
|
|
⚪
|
fomichev3000
approved these changes
Jun 13, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.