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

Performance issues #4382

Closed
acnetj opened this issue Dec 28, 2019 · 5 comments
Closed

Performance issues #4382

acnetj opened this issue Dec 28, 2019 · 5 comments
Labels
bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error

Comments

@acnetj
Copy link

acnetj commented Dec 28, 2019

Setup and configuration

MediaWiki | 1.34.0
PHP | 7.4.1 (fpm-fcgi)
MySQL | 5.7.25-0ubuntu0.16.04.2
Semantic MediaWiki | 3.1.1

After upgrade I have to save the page twice in order for the edits to show up on query results. If I just save once after edit the query output is still old. Wasn't that way before upgrade.

@JeroenDeDauw JeroenDeDauw added the bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error label Dec 28, 2019
@mwjames
Copy link
Contributor

mwjames commented Jan 4, 2020 via email

@kghbln
Copy link
Member

kghbln commented Jan 6, 2020

I have mixed experience and the result is probably dependent on the result format used. On sandbox I have to add the subobject and purge to get it to show in the graph (MW 1.34, SMW 3.1, PHP 7.2). On another wiki (MW 1.31, SMW 3.1 and PHP 7.0) the results show immediately for the table format.

@kghbln
Copy link
Member

kghbln commented Jan 6, 2020

Ok, I created some examples on sandbox and the behavior is identical for both formats. 1. Add subobject to add data, 2. Purge once to get the results. For me this is kinda expected and I think the @deferred query marker is meant to be the way out here.

When it comes to my previous #4382 (comment) the difference in the setup is MediaWiki and PHP however not SMW.

@kghbln kghbln added question and removed bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error labels Jan 6, 2020
@kghbln
Copy link
Member

kghbln commented Jan 8, 2020

Just saw that the @deferred query marker is probably only interacting with certain result printers.

@mwjames
Copy link
Contributor

mwjames commented Jan 26, 2020

I suspect this to be fixed by #4470.

@mwjames mwjames closed this as completed Jan 26, 2020
@kghbln kghbln added bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error and removed question labels Jan 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error
Projects
None yet
Development

No branches or pull requests

4 participants