Tyler Helmuth

icon company @honeycombio icon location Colorado

Results 799 comments of Tyler Helmuth

[pdata] Revisit need of `Val` suffix of `Value` methods

For Approach B part 2, I am missing the risk of ```go func (ms NumberDataPoint) GetDouble() float64 func (ms NumberDataPoint) SetDouble(v float64) func (ms NumberDataPoint) GetInt() int64 func (ms NumberDataPoint)...

[pdata] Revisit need of `Val` suffix of `Value` methods

I like Str. would the final API look like: ```go func (v Value) Type() ValueType func (v Value) GetStr() string func (v Value) SetStr(val) func (v Value) GetBool() bool func...

[pdata] Revisit need of `Val` suffix of `Value` methods

@dmitryax sorry, meant to remove all the Gets. That's what I get for multitasking.

[opentelemetry-collector] Refactoring labels

@nlamirault please break changes for different charts into separate PRs.

[pkg/opentelemetrytransformationlanguage] Rename TQL to OTTL

Not opposed to shortening the package directory to `oteltransformationlanguage`

[pkg/opentelemetrytransformationlanguage] Rename TQL to OTTL

@evan-bradley I think I addressed all your feedback.

[pkg/opentelemetrytransformationlanguage] Rename TQL to OTTL

@evan-bradley @bogdandrutu @kentquirk @kovrus please take a look.

[opentelemetry-demo] Refactor component env vars to better handle service names

/cc @wph95

[opentelemetry-demo] Refactor component env vars to better handle service names

@2start would you like to take point on this issue?

[chore] add chloggen for changelog

I'd vote for `unreleased_log` (or `unreleased_logs` or `unrelease_chlog`) in both repos. Something that indicates these are future changelog entries.