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

Create external scoped contexts #342

Closed
azaroth42 opened this issue Mar 25, 2020 · 1 comment
Closed

Create external scoped contexts #342

azaroth42 opened this issue Mar 25, 2020 · 1 comment
Assignees
@azaroth42
Labels
API The issue is about an API or service enhancement The issue describes an improvement to existing functionality or new functionality website The issue is about the website behavior, not the technical discussions wontfix The issue is valid, but for explained reasons won't be acted upon
Projects
Process
Milestone
1.0

Comments

@azaroth42
Copy link
Collaborator

As then there will be more cache hits, and reduces the size of the context.

So instead of being inline for Period, and Event, and Activity, and ... then it would be an external URI that can be loaded once (each) and then cached for all of the other classes. This reduces the readability slightly, but the performance improvement would be high with caching the processed context states.

Should be tested with different implementations before committing to this.

(Thanks to @dlongley, @gkellogg, @kasei)
@azaroth42 azaroth42 added enhancement The issue describes an improvement to existing functionality or new functionality API The issue is about an API or service labels Mar 25, 2020
@azaroth42 azaroth42 self-assigned this Mar 25, 2020
@azaroth42 azaroth42 added this to Website, Process, etc in Process Apr 7, 2020
@azaroth42 azaroth42 moved this from Website, Process, etc to API in Process Apr 7, 2020
@azaroth42 azaroth42 added the website The issue is about the website behavior, not the technical discussions label Aug 12, 2020
@azaroth42 azaroth42 added this to the 1.0 milestone Nov 15, 2023
@azaroth42 azaroth42 added the wontfix The issue is valid, but for explained reasons won't be acted upon label Apr 5, 2024
@azaroth42
Copy link
Collaborator Author

Readability is more important.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@azaroth42 azaroth42

Labels
API The issue is about an API or service enhancement The issue describes an improvement to existing functionality or new functionality website The issue is about the website behavior, not the technical discussions wontfix The issue is valid, but for explained reasons won't be acted upon
Projects
Process
  
API
Milestone
1.0
Development

No branches or pull requests
1 participant
@azaroth42