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

Enhancement: locally cache EAD XSLT results for performance #69

Open
ctgraham opened this issue Sep 7, 2021 · 1 comment · May be fixed by #76
Open

Enhancement: locally cache EAD XSLT results for performance #69

ctgraham opened this issue Sep 7, 2021 · 1 comment · May be fixed by #76

Comments

@ctgraham
Copy link

ctgraham commented Sep 7, 2021

Large finding aids can initiate many inline PHP calls and can be a performance hit. An optional local cache of the XSLT results per Drupal site could improve performance.

It is tempting to make an HTML Fedora derivative of the EAD datastream and serve that, but this misses the possibility of serving the same object from multiple Islandora frontends (and thus multiple base urls), and misses the changes which might come from checking at compile time for the existence of related objects in the links.
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 9, 2021
@ctgraham
discoverygarden#69: add option to cache EAD HTML transformations
5201ccf
@ctgraham ctgraham linked a pull request Sep 9, 2021 that will close this issue
Open
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 14, 2021
@ctgraham
discoverygarden#69: fix empty EAD HTML caching value
336029d
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 16, 2021
@ctgraham
discoverygarden#69: avoid unset variable warning
2c31472
@ctgraham
Copy link
Author

ctgraham commented Oct 5, 2021

N.b.: large finding aids may naturally exceed the size limit for cached data in default Drupal installs. (1MB for default memcache and mysql packet size configs). Perhaps this should be stored as temporary files instead.

@ctgraham ctgraham mentioned this issue Nov 1, 2021
Open
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Nov 2, 2021
@ctgraham
discoverygarden#69: add option to cache EAD HTML transformations to disk
8ba48fa
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 12, 2022
@ctgraham
discoverygarden#69: check for cron lock during caching
780d758
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 19, 2022
@ctgraham
discoverygarden#69: Use an expired cachefile until a new one is built
61ca9e3
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 20, 2022
@ctgraham
discoverygarden#69: correct expired cachfile path
faef4c8
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 20, 2022
@ctgraham
discoverygarden#69: improve expired cachefile management
cf23397
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 21, 2022
@ctgraham
discoverygarden#69: reuse expired cachefiles while new cache is being…
49836c9 
… generated and expire cachfiles based on EAD creation date
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 22, 2022
@ctgraham
discoverygarden#69: documentation cleanup
cf14143
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Sep 22, 2022
@ctgraham
discoverygarden#69: Use an expired cachefile until a new one is built…
8a61b09 
…, check for cron lock during caching
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Oct 4, 2022
@ctgraham
discoverygarden#69: clear expired cachefiles and check object existen…
84df877 
…ce when caching
ctgraham added a commit to ulsdevteam/islandora_solution_pack_manuscript that referenced this issue Oct 5, 2022
@ctgraham
discoverygarden#69: avoid re-using expired cachefile in cron
8c2778b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add option to cache EAD HTML transformations ulsdevteam/islandora_solution_pack_manuscript
1 participant
@ctgraham