<table><tr><td style="">dfaure requested changes to this revision.<br />dfaure added a comment.<br />This revision now requires changes to proceed.
</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/D8999" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I don't think it's KJob's job (haha) to be responsible for compressing change notifications, which is basically the use case you're mentioning. This can be done with a QTimer and an associate container of pending changes, on the application side. Such a solution is much more flexible because you can add the data you need (for fast lookups, for more precise decision making etc.) in that container, rather than having to introspect not-started-yet jobs. Not to mention the issue with in-progress jobs.</p>

<p>The pipeline here is KDirWatch ---> notification compression ---> actual operations that *should* be triggered = KJob.</p>

<p>Misusing KJob as a holder for notification compression seems wrong to me, there are better data structures for that.</p>

<p>-1 from me.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8999" rel="noreferrer">https://phabricator.kde.org/D8999</a></div></div><br /><div><strong>To: </strong>rjvbb, dfaure<br /><strong>Cc: </strong>apol, anthonyfieroni, Frameworks<br /></div>