filodb / FiloDB

Distributed Prometheus time series database
Apache License 2.0
1.43k stars 225 forks source link

fix(query): correcly escape LogicalPlan-to-String backslashes #1732

Open alextheimer opened 7 months ago

alextheimer commented 7 months ago

Pull Request checklist

Currently, escape characters themselves may not be correctly escaped when LogicalPlans are converted to Strings. This may cause e.g. parse errors when plans are converted to Strings then sent remotely. This PR adds the correct escape characters to all miscellaneous function args; these may contain regex and require backslash escapes.