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

Fix double query execution #3836

Merged
merged 1 commit into from Jul 2, 2019
Merged

Fix double query execution #3836

merged 1 commit into from Jul 2, 2019

Commits on Jun 27, 2019

  1. Fix double query execution 

    Our monitoring list views call SimpleQuery::hasResult() first in
order to determine whether there are results to display. This calls
fetchRow() which executes the underlying query the first time. If there
are resulsts, the query is iterated which executes the query again.
With this patch, SimpleQuery::hasResult() makes use of the inner
iterator instead of calling fetchRow(). The query is now executed only
once.
    @lippserd
    lippserd committed Jun 27, 2019
    Copy the full SHA
    b20291a View commit details
    Browse the repository at this point in the history