Skip to content

Commit 0a2dcee

Browse files
Parship Chowdhuryyurishkuro
andauthored
Remove sanitizer/service_name_sanitizer (dead code) (#7366)
## Which problem is this PR solving? - Resolves part of #7261 ## Description of the changes - When introduced: PR #67 - when replaced: - First Replacement - PR #3631 - modern OTLP implementation - PR #6077 - final location - PR #7158 - Current State: - Old-dead `cmd/collector/app/sanitizer/service_name_sanitizer.go` - Legacy-used `cmd/collector/app/sanitizer/empty_service_name_sanitizer.go` - Modern-used `internal/jptrace/sanitizer/emptyservicename.go` - No prod usage - Files to remove: - `cmd/collector/app/sanitizer/service_name_sanitizer.go` - `cmd/collector/app/sanitizer/service_name_sanitizer_test.go` ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [ ] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `npm run lint` and `npm run test` Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
1 parent 13e6b4a commit 0a2dcee

File tree

0 file changed

+0
-0
lines changed

    0 file changed

    +0
    -0
    lines changed

    0 commit comments

    Comments
     (0)