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

[Logs UI] Load log stream entries using async searches #86439

Open
5 of 10 tasks
weltenwort opened this issue Dec 18, 2020 · 4 comments
Open
5 of 10 tasks

[Logs UI] Load log stream entries using async searches #86439

weltenwort opened this issue Dec 18, 2020 · 4 comments
Assignees
Labels
Feature:Logs UI Logs UI feature Team:obs-ux-logs Observability Logs User Experience Team v7.12.0

Comments

@weltenwort
Copy link
Member

weltenwort commented Dec 18, 2020

Summary

As part of the make-it-slow effort (and thereby indirectly searchable snapshots and runtime fields) we want to use Elasticsearch's async search to execute potentially expensive queries. The queries to load the log entries in the log stream and the minimap summary certainly qualify as such.

related to #86441

Acceptance criteria

  • A search strategy for loading intervals of log stream entries exists.
  • A search strategy for loading log stream summary data exists. (probably not needed, just use the ESE search stategy directly)
  • The log stream entries are loaded using the corresponding search strategy.
  • The log entries in the "view-in-context" modals are loaded using the corresponding search strategy.
  • The log stream entries in the embeddable <LogStream /> component are loaded using the corresponding search strategy.
  • The log stream summary is loaded using the corresponding search strategy.
  • The request for log entries are cancelled when newer requests are issued.
  • The request for the log summary are cancelled when newer requests are issued.
  • The log entry requests can be explicitly cancelled and retried by the user.
  • The previous HTTP routes are not present anymore if no other usage remains.
@weltenwort weltenwort added Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v7.12.0 labels Dec 18, 2020
@weltenwort weltenwort added this to the Logs UI 7.12 milestone Dec 18, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@afgomez
Copy link
Contributor

afgomez commented Dec 21, 2020

Should this also affect the shared <LogStream /> component?


Edit, I re-read the description and I see the intention is to do it as part of this issue as well.

@weltenwort
Copy link
Member Author

Oops, I messed up escaping the < in <LogStream />, so it wasn't rendered correctly in the description - sorry about that 🙈

@weltenwort weltenwort self-assigned this Jan 13, 2021
@jasonrhodes jasonrhodes removed this from the Logs UI 7.12 milestone Jul 12, 2021
@sophiec20 sophiec20 removed the On Deck label Apr 24, 2023
@gbamparop gbamparop added Team:obs-ux-logs Observability Logs User Experience Team and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Nov 9, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

@botelastic botelastic bot added needs-team Issues missing a team label and removed needs-team Issues missing a team label labels Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Logs UI Logs UI feature Team:obs-ux-logs Observability Logs User Experience Team v7.12.0
Projects
None yet
Development

No branches or pull requests

6 participants