Skip to content
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

query: /graph unusable on large __name__ set due to autocomplete #2361

Closed
clhain opened this issue Apr 1, 2020 · 24 comments
Closed

query: /graph unusable on large __name__ set due to autocomplete #2361

clhain opened this issue Apr 1, 2020 · 24 comments

Comments

@clhain
Copy link

clhain commented Apr 1, 2020

thanos version: 0.11.0

We run a query instance that has a very large set of unique metric names (we're working to reduce them). Attempting to use the GUI at /graph causes the browser to go unresponsive for a long time as it grabs the list of values (a few seconds) and then renders the dropdown (a few dozen seconds). During that time the window is unresponsive.

Grafana helpfully disables the autocomplete feature on large sets, is it possible to do something similar here?

@GiedriusS
Copy link
Member

There is this: #2150. Just need someone to follow up and rebase on master.

@stale
Copy link

stale bot commented May 2, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label May 2, 2020
@bwplotka
Copy link
Member

bwplotka commented May 4, 2020

Still a problem.

@stale stale bot removed the stale label May 4, 2020
@bwplotka
Copy link
Member

bwplotka commented May 4, 2020

And something to tackle on React UI

cc @prmsrswt

@stale
Copy link

stale bot commented Jun 3, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jun 3, 2020
@bwplotka
Copy link
Member

bwplotka commented Jun 3, 2020 via email

@stale stale bot removed the stale label Jun 3, 2020
@stale
Copy link

stale bot commented Jul 3, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jul 3, 2020
@onprem
Copy link
Member

onprem commented Jul 3, 2020

@clhain can you check if it's still a problem with the new React UI? We limit the dom rendering to 100 items at max, so it's shouldn't be a problem now.

https://github.com/thanos-io/thanos/blob/master/pkg/ui/react-app/src/pages/graph/ExpressionInput.tsx#L89

@stale stale bot removed the stale label Jul 3, 2020
@clhain
Copy link
Author

clhain commented Jul 24, 2020

Hey, sorry I missed this update. I had a look at it in .13 and it's still unusable. Could be the match function running slowly as opposed to the actual rendering? Let me know if there's something else that would help debug, and thanks.

@GiedriusS
Copy link
Member

If it's still laggy then you must have so many metrics. label_names and label_values APIs now support time limits so maybe that could be leveraged in the future? 🤔

@stale
Copy link

stale bot commented Aug 23, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Aug 23, 2020
@stale
Copy link

stale bot commented Aug 30, 2020

Closing for now as promised, let us know if you need this to be reopened! 🤗

@stale stale bot closed this as completed Aug 30, 2020
@ipstatic
Copy link
Contributor

ipstatic commented Sep 9, 2020

This is still an issue, even with thew new interface.

@stale
Copy link

stale bot commented Nov 9, 2020

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Nov 9, 2020
@jameskranz
Copy link

This is still an issue, although #3381 helps mitigate the issue. It makes thanos usable with huge metric sets, but means autocomplete is unavailable.

@stale stale bot removed the stale label Nov 9, 2020
@stale
Copy link

stale bot commented Jan 9, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jan 9, 2021
@CNG
Copy link

CNG commented Jan 11, 2021

Still an issue.

@stale stale bot removed the stale label Jan 11, 2021
@stale
Copy link

stale bot commented Mar 16, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Mar 16, 2021
@stale
Copy link

stale bot commented Apr 7, 2021

Closing for now as promised, let us know if you need this to be reopened! 🤗

@stale stale bot closed this as completed Apr 7, 2021
@onprem
Copy link
Member

onprem commented Apr 8, 2021

Still valid. There's #4030 which switches the autocomplete with a codemirror based editor. We need to check if the performance is better or worse with the new editor enabled.

@onprem onprem reopened this Apr 8, 2021
@stale stale bot removed the stale label Apr 8, 2021
@Nexucis
Copy link
Contributor

Nexucis commented Apr 8, 2021

I tested with ~150k metrics. Works well, it's just a bit slow due to the network mainly. It would maybe need a way to warm the cache embedded in the PromQL editor

@stale
Copy link

stale bot commented Jun 9, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jun 9, 2021
@yeya24
Copy link
Contributor

yeya24 commented Jun 9, 2021

Looks like we can close this issue if it works well with the new editor.

@stale stale bot removed the stale label Jun 9, 2021
@yeya24 yeya24 closed this as completed Jun 9, 2021
@kalebris
Copy link

This still seems to be a problem. It seems to be working much better on Firefox, but on Chrome we are still seeing huge delays due to autocomplete. After typing in 1 character the ui freezes up for a good while (30 seconds or so) before the options appear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests