-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extended PromQL library? #56
Comments
I'm glad you are eager to add extended PromQL support to Promxy! The extended PromQL library code is located at app/vmselect/promql. It has a few dependencies on other bits of VictoriaMetrics, so it cannot be used as is in external projects. But it should be quite easy to fork the library and remove these dependencies there. While you are here, could you help with #52 ? |
I'll take a look at the promql thing likely this weekend. I'm not sure how
tightly coupled it is, but assuming I can pull it out into something
standalone would you be interested in that?
…On Thu, Jun 6, 2019, 8:50 AM Aliaksandr Valialkin ***@***.***> wrote:
I'm glad you are eager to add extended PromQL support to Promxy!
The extended PromQL library code is located at app/vmselect/promql
<https://github.com/VictoriaMetrics/VictoriaMetrics/tree/fb140eda33ccbe8fdea366c90cbcd0fef145b121/app/vmselect/promql>.
It has a few dependencies on other bits of VictoriaMetrics, so it cannot be
used as is in external projects. But it should be quite easy to fork the
library and remove these dependencies there.
While you are here, could you help with #52
<#52> ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#56?email_source=notifications&email_token=AAMLPHNM7STKIXXUSWG2Q4TPZEW5TA5CNFSM4HVDCZIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXDJRHI#issuecomment-499554461>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMLPHJTI7PO6TDQ3SU5CIDPZEW5TANCNFSM4HVDCZIA>
.
|
Yes! That would be great! The lib is almost self-contained with a few exceptions like integration with VictoriaMetrics storage. This part may be abstracted out in the standalone version. Other dependencies on VictioraMetrics code may be just removed from standalone version. |
We've started using VictoriaMetrics and need a HA setup. We've chosen promxy, but we lost the extended query language support which I'd love to be able to use. I haven't looked at how promxy parses queries, but I would assume since VM's parser is prometheus-compatible, it could use it (if it's easily extractable, which seems to be the case). Anyway, this comment is more like a 👍 than anything. Hope to see this soon! |
@jacksontj , MetricsQL parser (new name for Extended PromQL) has been extracted into a standalone library with the minimum number of external dependencies - Closing this feature request. |
Hello, the author of Promxy here. First off, this definitely looks like an interesting project and I'm excited to see where it goes. Also, I do appreciate the link over to promxy -- I'm glad I can be of help.
While reading through the docs for VictoriaMetrics I see mention of this Extended PromQL functionality. I did a quick search through the code and didn't see a replacement promql engine (TBH I spent about 5m-- so it might be there and I missed it). If there is a library for usage I could look into optionally adding support for this promql Extenstion into Promxy.
The text was updated successfully, but these errors were encountered: