Skip to content

Conversation

@loganharbour
Copy link
Member

@loganharbour loganharbour commented Nov 3, 2025

Closes #31831

This is in support of performance improvements for the web application that will parse these databases.

  • Changes all arguments for getting a test to a TestName instead of two strings (folder and test name)
  • Add query_test for getting a test if it is available, otherwise None
  • Do not construct each individual test within a single result on init (speeds up initial construction)
  • Optimize serialize and deserialize methods
  • Don't connect to the client/database unless needed

@loganharbour loganharbour self-assigned this Nov 3, 2025
@moosebuild
Copy link
Contributor

moosebuild commented Nov 3, 2025

Job Documentation, step Docs: sync website on 8fd75ad wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

moosebuild commented Nov 3, 2025

Job Coverage, step Generate coverage on 8fd75ad wanted to post the following:

Framework coverage

Coverage did not change

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

@loganharbour loganharbour force-pushed the resultsstore_serialize branch from 1bc2c70 to a55f435 Compare November 14, 2025 22:11
@loganharbour loganharbour force-pushed the resultsstore_serialize branch from 8d32873 to aa89948 Compare November 14, 2025 23:46
@moosebuild
Copy link
Contributor

Job Test, step Results summary on 8fd75ad wanted to post the following:

Framework test summary

Compared against b79a48d in job civet.inl.gov/job/3380566.

No change

Modules test summary

Compared against b79a48d in job civet.inl.gov/job/3380566.

No change

@loganharbour loganharbour force-pushed the resultsstore_serialize branch from cee50d4 to 9a37cc2 Compare November 15, 2025 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Speed up resultsstore serialization and deserialization

2 participants