<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/106711/">http://git.reviewboard.kde.org/r/106711/</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 68c6b9345f19da58bf384540354b556f4208d594 by Vishesh Handa to branch master.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On October 3rd, 2012, 12:11 p.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, Christian Mollekopf and Sebastian Trueg.</div>
<div>By Vishesh Handa.</div>


<p style="color: grey;"><i>Updated Oct. 3, 2012, 12:11 p.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;">    StoreResources: Add a flag to force duplicate detection in the graph
    
    By default each SimpleResource in the graph was always hash (an
    expensive process) and then checked for duplicates with the other
    SimpleResources in the graph.
    
    This feature was only added cause the PIM guys were pushing large
    quantities of duplicate data. It doesn't make sense for everyone to pay
    the penalty for one application.
    
    They can enable this feature with the MergeDuplicateResources flag.
</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Updated the relevant tests</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>libnepomukcore/datamanagement/datamanagement.h <span style="color: grey">(2ac60a5)</span></li>

 <li>services/storage/datamanagementmodel.cpp <span style="color: grey">(7c05cfd)</span></li>

 <li>services/storage/test/datamanagementmodeltest.cpp <span style="color: grey">(3d3340c)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/106711/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>