[Nepomuk] Review Request: Client: Cache the current bindings
Commit Hook
null at kde.org
Wed Aug 29 16:51:49 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106231/#review18260
-----------------------------------------------------------
This review has been submitted with commit f4b1d9413c415871bae256ebb1bc9dee891eee70 by Vishesh Handa to branch 2.8.
- Commit Hook
On Aug. 27, 2012, 10:22 a.m., Vishesh Handa wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106231/
> -----------------------------------------------------------
>
> (Updated Aug. 27, 2012, 10:22 a.m.)
>
>
> Review request for Nepomuk, Soprano, Aleix Pol Gonzalez, and Sebastian Trueg.
>
>
> Description
> -------
>
> Client: Cache the current bindings
>
> Cache the current bindings when running the next() in a local variable.
> This way we avoid asking for it during each operator[] and current.
>
> Results in a MASSIVE performance boost - 6 secs vs 1 sec on a large
> query which had a lot of bindings.
>
>
> Diffs
> -----
>
> client/clientqueryresultiteratorbackend.h 2d10756
> client/clientqueryresultiteratorbackend.cpp babbdcc
>
> Diff: http://git.reviewboard.kde.org/r/106231/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Vishesh Handa
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20120829/6257de02/attachment.html>
More information about the Nepomuk
mailing list