filodb / FiloDB

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

fix(query) misleading message when label filter is too long. #1759

Closed yu-shipit closed 6 months ago

yu-shipit commented 6 months ago

Pull Request checklist

Current behavior :

Error code does not indicate that the regex is too long. valid_query: Cannot parse […]

New behavior : Error code will indicate that the regex is too long. The error will contain meaningful message Regular expression filters should be <=

BREAKING CHANGES

If this PR contains a breaking change, please describe the impact and migration path for existing applications. If not please remove this section.

Breaking changes may include:

Other information: