opentelemetry-collector: Return an error on `config.Map.Set`

seems there’s at least a couple of calls that could trigger an error in this func with Load just above this line, I would be in favour of returning an error from Set

_Originally posted by @codeboten in https://github.com/open-telemetry/opentelemetry-collector/pull/4462#discussion_r753349403_

About this issue

  • Original URL
  • State: closed
  • Created 3 years ago
  • Comments: 15 (15 by maintainers)

Most upvoted comments

@jpkrohling yes, I think this should also be required for GA since it’s a breaking change