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

fix: improve preview speed on Stardog >= 10 #1662

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

Rdataflow
Copy link
Contributor

wrt Stardog issue 2094
see also SBAR-1122

@bprusinowski can you please push this urgent workaround to the upcoming release? THX
Context: The goal is to mitigate unusual long preview query times i.e. preview on Strompreise would take ~60s otherwise :/

cc @sosiology @adintegra

wrt Stardog issue 2094
see also SBAR-1122
Copy link

vercel bot commented Jul 17, 2024

@Rdataflow is attempting to deploy a commit to the Interactive Things Team on Vercel.

A member of the Team first needs to authorize it.

@Rdataflow
Copy link
Contributor Author

@bprusinowski can you please push this urgent workaround to the upcoming release? THX

cc @ortnever this should help and work around the slowness you just observed in preview - you may push to have this in the release

@adintegra
Copy link
Contributor

@Rdataflow thanks for reaching out. As you're aware, we currently don't have active resources on the project. Nonetheless, we'll try and get this workaround integrated before the upcoming PROD release.
/cc @sosiology

@bprusinowski bprusinowski merged commit 08ee34b into visualize-admin:main Jul 18, 2024
1 of 2 checks passed
@Rdataflow
Copy link
Contributor Author

@adintegra thanks we are aware - that's why I only ask for this minimal change 👍
@bprusinowski THX 🎉

@Rdataflow Rdataflow deleted the patch-2 branch July 18, 2024 10:19
@ortnever
Copy link

Thank you all (@Rdataflow @adintegra @bprusinowski ) for reacting so quickly to fix this performance problem. Deployment on PROD is scheduled for the first week of August, so this will be part of it. Great !

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.

4 participants