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

Table performance issue when inserting new rows #4284

Closed
2 of 5 tasks
akhenry opened this issue Sep 30, 2021 · 2 comments
Closed
2 of 5 tasks

Table performance issue when inserting new rows #4284

akhenry opened this issue Sep 30, 2021 · 2 comments

Comments

@akhenry
Copy link
Contributor

akhenry commented Sep 30, 2021

Summary

Tables are exhibiting slow performance on modest datasets (1000+ rows). This becomes particularly apparent with telemetry that has large data objects, such as Yamcs "Events" data.

The issue appears to be in the check for duplicate values.

The code will first create a shortlist of candidate duplicates based on timestamp alone. It will then do a more exhaustive search using _.isEqual on this candidate list to find genuine duplicates. From performance profiling I am seeing this function being called with extremely high frequency (see screenshot below), which is unexpected.

The first thing to do is to just confirm that the code that creates the shortlist of potential duplicates is working as expected and that it's not accidentally scanning over the entire telemetry set, because it's surprising to see the duplicate check running so regularly. If this is not the issue then we need to consider whether there is a more performant method of detecting duplicates than _.isEqual.

Screen Shot 2021-09-30 at 4 16 37 PM

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?
@akhenry
Copy link
Contributor Author

akhenry commented Sep 30, 2021

After some further investigation I suspect that this issue is the root of the problem.

@akhenry
Copy link
Contributor Author

akhenry commented Oct 4, 2021

Closing in favor of a corresponding issue in the Open MCT - Yamcs repo.

@akhenry akhenry closed this as completed Oct 4, 2021
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

2 participants