Skip to content

Commit

Permalink
Merge pull request #116 from keep94/sender-pinterface
Browse files Browse the repository at this point in the history
Make Sender so that it can't be implemented externally.
  • Loading branch information
keep94 authored Nov 8, 2022
2 parents 04d1795 + 2fcf215 commit e144509
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
4 changes: 4 additions & 0 deletions senders/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ type Sender interface {
EventSender
internal.Flusher
Close()
private()
}

type wavefrontSender struct {
Expand Down Expand Up @@ -74,6 +75,9 @@ func (sender *wavefrontSender) Start() {
sender.eventHandler.Start()
}

func (sender *wavefrontSender) private() {
}

func (sender *wavefrontSender) SendMetric(name string, value float64, ts int64, source string, tags map[string]string) error {
line, err := metricLine(name, value, ts, source, tags, sender.defaultSource)
if err != nil {
Expand Down
3 changes: 3 additions & 0 deletions senders/client_multi.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ func NewMultiSender(senders ...Sender) MultiSender {
return ms
}

func (ms *multiSender) private() {
}

func (ms *multiSender) SendMetric(name string, value float64, ts int64, source string, tags map[string]string) error {
var errors multiError
for _, sender := range ms.senders {
Expand Down
3 changes: 3 additions & 0 deletions senders/client_noop.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ func NewWavefrontNoOpClient() (Sender, error) {
return defaultNoopClient, nil
}

func (sender *wavefrontNoOpSender) private() {
}

func (sender *wavefrontNoOpSender) Start() {
// no-op
}
Expand Down

0 comments on commit e144509

Please sign in to comment.