Re: FLIP-401: Remove brackets around keys returned by MetricGroup#getAllVariables

2023-07-18 Thread Maximilian Michels
Hi Chesnay, +1 Sounds good to me! -Max On Tue, Jul 18, 2023 at 10:59 AM Chesnay Schepler wrote: > > MetricGroup#getAllVariables returns all variables associated with the > metric, for example: > > | = abcde| > | = ||0| > > The keys are surrounded by brackets for no particular reason. > > In vir

FLIP-401: Remove brackets around keys returned by MetricGroup#getAllVariables

2023-07-18 Thread Chesnay Schepler
MetricGroup#getAllVariables returns all variables associated with the metric, for example: | = abcde| | = ||0| The keys are surrounded by brackets for no particular reason. In virtually every use-case for this method the user is stripping the brackets from keys, as done in: * our datadog r