<table><tr><td style="">bruns added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28682">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D28682#644430" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D28682#644430</a>, <a href="https://phabricator.kde.org/p/astippich/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@astippich</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D28682#644414" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D28682#644414</a>, <a href="https://phabricator.kde.org/p/bruns/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@bruns</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>In general fine for me.</p>

<p>How will Elisa deal with the indexer maybe crashing inbetween? Do we also need a signal for a batch start?</p></div>
</blockquote>

<p>From my (limited) understanding, the finishedIndexingFile will no be emitted when an indexers crashes, does it? Then I think it is fine without an additional signal as long as the done() signal is still emitted as it is possible to reset the applications' internal list. Or will it retry some files of the current batch?</p></div>
</blockquote>

<p>It will not be emitted for the file it crashed on, but for the earlier files. So you may have the following for the files 'a, b, c, d':</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">S:a, F:a</li>
<li class="remarkup-list-item">S:b, F:b</li>
<li class="remarkup-list-item">S:c <crash></li>
<li class="remarkup-list-item">S:a, F:a</li>
<li class="remarkup-list-item">S:b, F:b</li>
<li class="remarkup-list-item">done</li>
<li class="remarkup-list-item">S:c, <crash></li>
<li class="remarkup-list-item">S:d, F:d</li>
<li class="remarkup-list-item">done</li>
</ul>

<p>So when you only listen to <tt style="background: #ebebeb; font-size: 13px;">finished</tt> and <tt style="background: #ebebeb; font-size: 13px;">done</tt>, you should keep the files in a <tt style="background: #ebebeb; font-size: 13px;">set</tt>, but after the <tt style="background: #ebebeb; font-size: 13px;">done</tt> the earlier files should be committed.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28682">https://phabricator.kde.org/D28682</a></div></div><br /><div><strong>To: </strong>astippich, Baloo, bruns<br /><strong>Cc: </strong>mgallien, kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>