carbonapi icon indicating copy to clipboard operation
carbonapi copied to clipboard

" is a valid char in metric names

Open dgryski opened this issue 11 years ago • 2 comments

Figure out the full set. Figure out which subset we want to support. Hope we don't have to fully break expression parsing in the meantime.

dgryski avatar Oct 17 '14 11:10 dgryski

A read-through of the graphite grammar gives me: ASCII 33 thru 126 (pyparsing printables):

!"#$%&'()*+,-./
0123456789
:;<=>?@
ABCDEFGHIJKLMNOPQRSTUVWXYZ
[\]^_`
abcdefghijklmnopqrstuvwxyz
{|}~

While these symbols must be \ escaped if present as part of the metric name:

(){},=.'"\

/cc @kanatohodets

nnuss avatar Aug 28 '16 09:08 nnuss

Do we need any extra work in this place? Or we should close this and improve handling of the cases mentioned in #175?

Civil avatar Apr 26 '17 07:04 Civil