<p dir="ltr">Hmm I wonder what the regression in the SQL server driver is. I added a is valid check although that should have low overhead.</p>
<br><div class="gmail_quote"><div dir="ltr">On Thu, 25 May 2017, 9:49 AM Mathieu Pellerin <<a href="mailto:nirvn.asia@gmail.com">nirvn.asia@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>On the 3.0 vs 2.x load time issue, there's a one time operation we do in 3.0 which we didn't do earlier, namely the creation of the saved style database (ie import of styles stored in symbology-ng.xml).</div><div dir="auto"><br></div><div dir="auto">As I mentioned above, it's a one time operation, so the load time cost is only occuring upon launching QGIS with an empty user profile directory. Subsequent load times aren't impacted.</div><div dir="auto"><br></div><div dir="auto">On my machine, the import takes 650ms (or actually less, I can't remember).</div></div><div dir="auto"><div dir="auto"><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On May 25, 2017 5:50 AM, "Nyall Dawson" <<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>> wrote:<br type="attribution"><blockquote class="m_-6408945023133182499quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_-6408945023133182499quoted-text">On 25 May 2017 at 02:20, Giovanni Manghi <<a href="mailto:giovanni.manghi@gmail.com" target="_blank">giovanni.manghi@gmail.com</a>> wrote:<br>
> Hi Nyall<br>
><br>
><br>
><br>
>> - it's not possible to directly compare the repo provided versions vs<br>
>> self-compiled builds. They'll be built using different compilers with<br>
>> different optimisation levels. Are you able to re-run the tests which<br>
>> used the repo builds with self-compiled builds instead?<br>
><br>
><br>
> I compiled tag final-2_18_8 and I confirm that with "time" results<br>
> (start qgis + loading a large dataset) are basically the same as the<br>
> ones obtained by compiling the 2.18 branch.<br>
<br>
</div>Great - thanks for the confirmation. I've just finished another round<br>
of benchtesting and here's my conclusions:<br>
<br>
1. Attribute table performance is fixed and back to 2.14.0 speed.<br>
<br>
2. There's been no regressions in rendering speed or layer loading<br>
speed from 2.14.0 -> 2.18.8. Tested with a variety of layer geometry<br>
types & sizes on Ubuntu 16.04, using release builds. Only shapefiles<br>
and the OGR provider were tested.<br>
<br>
3. There's strong evidence that we have a severe performance<br>
regression just in the mssql provider<br>
(<a href="https://issues.qgis.org/issues/15752" rel="noreferrer" target="_blank">https://issues.qgis.org/issues/15752</a>). I believe that bug should be<br>
reopened but renamed to reflect that it affects mssql only. Note that<br>
I haven't confirmed this regression or run any benchtests on it yet.<br>
<br>
4. Giovanni's tests have revealed a regression in the load times for<br>
QGIS (as opposed to rendering/layer loading times). This is somewhat<br>
expected. The majority of QGIS' load time occurs with loading the<br>
processing plugin, and specifically with adding all the providers and<br>
algorithms. So given the vastly increased number of algorithms<br>
available in 2.18 vs 2.14, it's a natural (but unfortunate) side<br>
effect. I think we'll have to just live with this in 2.x. For 3.x we<br>
could probably move the algorithm/provider initialization to a<br>
background tasks which is kicked off only after the QGIS interface has<br>
loaded.<br>
<br>
Giovanni - do you agree with these conclusions?<br>
<font color="#888888"><br>
Nyall<br>
</font><div class="m_-6408945023133182499elided-text">_______________________________________________<br>
Qgis-psc mailing list<br>
<a href="mailto:Qgis-psc@lists.osgeo.org" target="_blank">Qgis-psc@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/qgis-psc" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-psc</a></div></blockquote></div><br></div></div></div>
_______________________________________________<br>
Qgis-psc mailing list<br>
<a href="mailto:Qgis-psc@lists.osgeo.org" target="_blank">Qgis-psc@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/qgis-psc" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-psc</a></blockquote></div>