<table><tr><td style="">smithjd 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/D11529">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/D11529#233550" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11529#233550</a>, <a href="https://phabricator.kde.org/p/michaelh/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@michaelh</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Under the premise that I still have to learn about baloo's inner workings, here are some concerns:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I'm not convinced, that index cleaning should be part of the dbus interface. Why should cleaning be done out of process? Do we really want any application to trigger database manipulation? Could you elaborate your rationale please?</li>
</ul></div>
</blockquote>

<p>IndexCleaner fits the current implementation of checking for file changes quite well.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Which application is using baloo dbus interface anyway and which functions? <a href="https://phabricator.kde.org/p/mgallien/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mgallien</a> How is elisa using it?</li>
<li class="remarkup-list-item">Purging stale entries is by far not enough to get the index in a good shape. As soon as removable drives or network shares are involved all sorts of weird things can happen. IndexCleaner is much too simple to account for that.</li>
</ul></blockquote>

<p>IndexCleaner is simple and can handle 99.9% of file deindexing requirements, and is now safe for use in combination with unmounted indexed volumes. IndexCleaner is a vital MIA part of Baloo. An additional purger could conceivably be useful in certain cicumstances, but given that the index is regeneratable data, I don't know that it's even warranted. This could be a good use for baloodb, leaving the generic automatic part to Baloo.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Looks like indexcleaner has been dead code until now. Do we need to be concerned about Vishesh's commit message? <a href="https://cgit.kde.org/baloo.git/commit/?id=ea2afe88b0c4299d7540e5b6c8b8e46858336f0c" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/baloo.git/commit/?id=ea2afe88b0c4299d7540e5b6c8b8e46858336f0c</a></li>
</ul></blockquote>

<p>As above, this already neatly fits into the workflow.</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/D11529">https://phabricator.kde.org/D11529</a></div></div><br /><div><strong>To: </strong>smithjd, Baloo, vhanda, michaelh<br /><strong>Cc: </strong>mgallien, Frameworks, ashaposhnikov, michaelh, astippich, spoorun, nicolasfella, ngraham, alexeymin<br /></div>