Skip to content

cache when looking for schemas#242

Merged
ejratl merged 4 commits intomasterfrom
improve-performance
Jul 18, 2025
Merged

cache when looking for schemas#242
ejratl merged 4 commits intomasterfrom
improve-performance

Conversation

@rpiazza
Copy link
Copy Markdown
Contributor

@rpiazza rpiazza commented Mar 15, 2025

Some of the examples for the incident extension were taking forever to validate.

Looking for schemas, especially $ref ones to github (e.g., timestamp) were being asked for time and time again.

I cache them in a mapping so they only have to be looked up once.

Similar for looking for extension schemas

@rpiazza rpiazza requested a review from chisholm March 15, 2025 20:56
@rpiazza
Copy link
Copy Markdown
Contributor Author

rpiazza commented Mar 15, 2025

Many of the tests are failing - But I don't know why - lots of "invalid" tests look invalid to me, so why don't they pass?

@rpiazza
Copy link
Copy Markdown
Contributor Author

rpiazza commented Mar 16, 2025

Ignore the previous note - my changes caused the errors, but now they are all fixed.

Copy link
Copy Markdown
Contributor

@ejratl ejratl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ejratl ejratl merged commit 3e24f8c into master Jul 18, 2025
13 checks passed
@ejratl ejratl deleted the improve-performance branch July 18, 2025 15:17
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.

2 participants