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.

Sign up for GitHub

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

Jump to bottom

[Infra UI] Defer field resolution in Data View powered views #139341

Closed
miltonhultgren opened this issue Aug 24, 2022 · 3 comments
Closed

[Infra UI] Defer field resolution in Data View powered views #139341

miltonhultgren opened this issue Aug 24, 2022 · 3 comments
Labels
blocked Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature needs-refinement A reason and acceptance criteria need to be defined for this issue Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@miltonhultgren
Copy link
Contributor

Blocked by #139340
Related #127616

AC

  • The Log View service path which relies on Data Views to resolve fields now defers field resolution to later
  • The Lens powered Hosts view in the Metrics UI now doesn't resolve fields at all
  • The Log View service path which doesn't rely on Data Views doesn't resolve fields with getFieldsForWildcard but defers to the UI to resolve fields using the index_fields endpoint

Notes

I don't know where to place to field resolution for the data views path of the Log View, perhaps the index_fields endpoint can be made to also support resolving fields from a data view?
@miltonhultgren miltonhultgren added Feature:Metrics UI Metrics UI feature 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 labels Aug 24, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

This was referenced Aug 24, 2022
Closed
Merged
@smith smith added the blocked label Aug 24, 2022
@smith smith added the needs-refinement A reason and acceptance criteria need to be defined for this issue label Sep 7, 2022
@smith
Copy link
Contributor

smith commented Sep 7, 2022

Taking this out of the refining column because it's blocked.

@miltonhultgren
Copy link
Contributor Author

Closing this for now, we'll wait and see how the improved _fields_caps API performs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocked Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature needs-refinement A reason and acceptance criteria need to be defined for this issue Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@smith @miltonhultgren @elasticmachine