Skip to content
Snippets Groups Projects
  • Ted Possible's avatar
    72b650d3
    7703 metricsql support metric names in quotes as alternative to name metric pattern (#40) · 72b650d3
    Ted Possible authored
    
    * Initial tests for label keys to be quoted
    
    * a few more tests involving numbers and quotes
    
    * Adds support for quoted UTF8 label names and special case label name without a value to specify metric name
    
    * Fixed bug with not checking for errors before calling isMetricNameFilter()
    
    * Added more testing for quoted labels
    Fixed bug with error not reporting because of isPossibleMetric being when a label name was not quoted
    
    * Minor tweaks to comments
    
    * Fixed tests to disregard two metric names
    Added tests to ensure setting two metric names threw errors
    
    * Removed extraneous comment
    
    * Duplicate metric names are now allowed
    Refactored code checking for quoted strings into isQuoteString
    Added some additional tests
    
    * Modified string output to be unescaped UTF8 chars
    
    * Revert "Modified string output to be unescaped UTF8 chars"
    
    This reverts commit 41dc7c1d42197a6debde2f41e622830e7439c692.
    
    * Reverted parser_example_test.go back to using standard format of metric filters
    
    Co-authored-by: default avatarRoman Khavronenko <roman@victoriametrics.com>
    
    * Removed / modified some tests to exclude the case of multiple metric names - this case will now throw an error and is tested in expected Fail cases
    
    * minor comments
    
    ---------
    
    Co-authored-by: default avatarRoman Khavronenko <roman@victoriametrics.com>
    7703 metricsql support metric names in quotes as alternative to name metric pattern (#40)
    Ted Possible authored
    
    * Initial tests for label keys to be quoted
    
    * a few more tests involving numbers and quotes
    
    * Adds support for quoted UTF8 label names and special case label name without a value to specify metric name
    
    * Fixed bug with not checking for errors before calling isMetricNameFilter()
    
    * Added more testing for quoted labels
    Fixed bug with error not reporting because of isPossibleMetric being when a label name was not quoted
    
    * Minor tweaks to comments
    
    * Fixed tests to disregard two metric names
    Added tests to ensure setting two metric names threw errors
    
    * Removed extraneous comment
    
    * Duplicate metric names are now allowed
    Refactored code checking for quoted strings into isQuoteString
    Added some additional tests
    
    * Modified string output to be unescaped UTF8 chars
    
    * Revert "Modified string output to be unescaped UTF8 chars"
    
    This reverts commit 41dc7c1d42197a6debde2f41e622830e7439c692.
    
    * Reverted parser_example_test.go back to using standard format of metric filters
    
    Co-authored-by: default avatarRoman Khavronenko <roman@victoriametrics.com>
    
    * Removed / modified some tests to exclude the case of multiple metric names - this case will now throw an error and is tested in expected Fail cases
    
    * minor comments
    
    ---------
    
    Co-authored-by: default avatarRoman Khavronenko <roman@victoriametrics.com>