Requirement
Summary
There is a small naming inconsistency in the remote sampling extension.
In cmd/jaeger/internal/extension/remotesampling/extension.go, the struct contains a field named strategyProvider of type samplingstrategy.Provider. A TODO comment in the code indicates that this field should be renamed to Provider (not a “store”) to better reflect its purpose and align with the actual type.
Motivation
- Improves naming clarity and consistency
- Avoids confusion between provider and store concepts (e.g.
adaptiveStore)
- Addresses existing acknowledged technical debt
Proposed Change
- Rename
strategyProvider to provider
- Update all internal references and related tests
Impact
This is a safe internal refactor:
- No public API changes
- No behavioral changes
- No configuration impact