<table><tr><td style="">leinir updated this revision to Diff 14080.<br />leinir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-osiymktm2asg4je/" rel="noreferrer">(Show Details)</a><br />leinir 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/D5639" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Simplify the logic, and only do the xmlloader caching, not the documents themselves which (as apol points out) should be cached already anyway. Not only that, but that codepath was never triggered when i rechecked it after implementing the xmlloader cache, and so, simpler logic is a good thing. Adjusted summary to match.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This is <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">a two layer</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">is a simple</span> approach to creating less network traffic when initialising <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">a</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">multiple instances of</span> KNSCore::Eng<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">ine:<br />
<br />
* Firstly,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ine in the same application.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">cache the xml documents themselves<br />
* Secondly</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Very simply</span>, <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">we </span>share XML fetch jobs when many are created at the same time for the same URL<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">.</span><br />
<br />
The result of this patch in a single fetch per provider url per application launch, and in connection with D5638 we will fetch the data from the network only once, until the cache is invalidated, all in all resulting in much less traffic and less hammering of the servers.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D5639?vs=13921&id=14080" rel="noreferrer">https://phabricator.kde.org/D5639?vs=13921&id=14080</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5639" rel="noreferrer">https://phabricator.kde.org/D5639</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/core/engine.cpp</div></div></div><br /><div><strong>To: </strong>leinir, whiting, apol<br /><strong>Cc: </strong>Frameworks<br /></div>