Small fixes - Githubissues

metrico / qryn

⭐️ All-in-One Polyglot Observability with OLAP Storage for Logs, Metrics, Traces & Profiles. Drop-in Grafana Cloud replacement compatible with Loki, Prometheus, Tempo, Pyroscope, Opentelemetry, Datadog and beyond :rocket:

https://qryn.dev

GNU Affero General Public License v3.0

1.24k stars 68 forks source link

Small fixes #426

Closed Diamondy4 closed 10 months ago

Diamondy4 commented 10 months ago

ClickHouse 22.10 removed support for the WITH TIMEOUT section for LIVE VIEW . Try to create temporary live view then drop it.
Missing new in matcher transpiler causes request to crash.
parseInt doesn't fail if there is letters, allowing garbage input.

CLAassistant commented 10 months ago

All committers have signed the CLA.

akvlad commented 10 months ago

@Diamondy4 thanks a lot for the fixes. One small change please and we gladly merge. Thanks a lot.

Diamondy4 commented 10 months ago

@akvlad Which change?

akvlad commented 10 months ago

@Diamondy4 Don't you see the review? Oh. GH needs another button to be pressed.

akvlad commented 10 months ago

Merged. Thanks a lot for the participation.