<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/110794/">http://git.reviewboard.kde.org/r/110794/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Nepomuk, Jörg Ehrichs and Vishesh Handa.</div>
<div>By Simeon Bird.</div>
<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;">FileIndexer: report if indexing certain files failed
At the moment, if the fileindeer chokes on a file, it will be repeatedly
requeued, and the indexer will consume all the CPU forever.
This patch stores a list of any files that did not index internally and
does not add them to the queue. This list does not persist across
restarts, because indexing failures may well be transient.
We are limited to 100 failed files. Once we have this many, just stop
indexing.
Furthermore add dbus methods to the indexer service to report the
number of failed files as well as their filenames and error message.
BUG: 315817
FIXED-IN: 4.11
There should be some sort of ui component for this, so that user can easily see and
copy-paste the names of files that didn't index properly. I didn't write this yet,
as I was hoping that by waiting I can just add it to the new qml controller.
I'm not completely convinced by the showFailedFiles function either - a newline
separated list seems like it could be improved upon.</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;">Compiled, ran, called the dbus methods. No files fail to index for me now though</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=315817">315817</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>interfaces/org.kde.nepomuk.FileIndexer.xml <span style="color: grey">(9e56cb139f05e4a4267adf575d0894c5a69935a1)</span></li>
<li>services/fileindexer/fileindexer.h <span style="color: grey">(3d87a04bcf6a7d8c9402e8085daeb7c630bc4e91)</span></li>
<li>services/fileindexer/fileindexer.cpp <span style="color: grey">(8c712dfc43af06c55f5503295e8460da89e5fcb5)</span></li>
<li>services/fileindexer/fileindexingqueue.h <span style="color: grey">(b017f226b0a209d57bcac7b9fb949d5b17e79cba)</span></li>
<li>services/fileindexer/fileindexingqueue.cpp <span style="color: grey">(2b119255e39fc873db08148291e1638e1a8c510a)</span></li>
<li>services/fileindexer/indexscheduler.h <span style="color: grey">(c6a9e35bed7ce885a8948aec37eccd5d23f7eb5d)</span></li>
<li>services/fileindexer/indexscheduler.cpp <span style="color: grey">(2835eb21274c80738b47f4526c47028d0d9dd06e)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110794/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>