Skip to content

tsm: narrow constructor input to a tsm.Config value type #1011

Description

@crandles

tsm's constructor reaches into o.TSMOptions, o.MaxCaptureBytes, o.MaxFanoutCaptureBytes, and o.OutputFormat from the full options struct in pkg/backends/alb/mech/tsm/time_series_merge.go. OutputFormat is provider metadata, not load-balancing behavior, so threading it through tsm construction couples a mechanism to backend-format knowledge.

Proposal: introduce a tsm.Config value type populated once at the registry layer, pass it into New() in place of the full options.

Metadata

Metadata

Assignees

Labels

albaffects the application load balancerrefactorOptimize or modernize existing codetsmergeissue affects Time Series Merge

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions