<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 />



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Brilliant.</pre>
 <br />







<p>- Sebastian</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>