<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/106474/">http://git.reviewboard.kde.org/r/106474/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit e966c39de0c708d4e23c4e730c1946e0d5849fdf by Vishesh Handa to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On September 17th, 2012, 10 a.m., Vishesh Handa wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Nepomuk and Sebastian Trueg.</div>
<div>By Vishesh Handa.</div>
<p style="color: grey;"><i>Updated Sept. 17, 2012, 10 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"> Use Virtuoso's inbuilt prefix expansion instead of the NRLModel
The NRL Model is terribly slow and consumes as high as 60% of the actual
time it takes to run some queries. Arguably, these queries were small
and virtuoso ran them quite fast. But it still is a lot of unnecessary
time spent on modifying the query. (Based on callgrind)
Since virtuoso already needs to parse the sparql query, it seems to make
more sense to let it do the job. Plus, it cleans up our code rather
nicely.
BUG: 304361
---
With this it is now faster to write queries purely in sparql and avoid the Node::resourceToN3 call which is rather expensive.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>services/storage/datamanagementadaptor.cpp <span style="color: grey">(499fcd0)</span></li>
<li>services/storage/repository.h <span style="color: grey">(63e4c5c)</span></li>
<li>services/storage/repository.cpp <span style="color: grey">(62b12f9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106474/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>