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

Multiple telemetries destroy table performance #5102

Closed
1 of 5 tasks
davetsay opened this issue Apr 22, 2022 · 1 comment · Fixed by #5042
Closed
1 of 5 tasks

Multiple telemetries destroy table performance #5102

davetsay opened this issue Apr 22, 2022 · 1 comment · Fixed by #5042
Assignees
Labels

Comments

@davetsay
Copy link
Contributor

Summary

For tables with multiple telemetries added, combining data into the table is not performant. The datasets are combined individually, and each additional dataset takes exponentially longer than the last.

Expected vs Current Behavior

Tables should load and render rows in a reasonable amount of time

Steps to Reproduce

  1. Create a telemetry table and add several telemetry endpoints (5 to 10)
  2. Query a time period (start small) using the time conductor (fixed time is easier to observe and control amount of datapoints)
  3. Observe time it takes for each dataset to render, shown by the rows count in the lower right of the table updating
  4. Change time window to increase amount of datapoints if time to render each set of data is negligible
  5. Can also compare to an overlay plot with same data for kicks

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS:
  • Browser:

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?

Additional Information

@khalidadil
Copy link
Contributor

Verified Fixed in Testathon on 04/25/2022

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

Successfully merging a pull request may close this issue.

4 participants