MSSQL Estimated table parameter is not used

[qgib]

Author Name: Jeremy Palmer (@palmerj)
Original Redmine Issue: 5252
Affected QGIS version: master
Redmine category:data_provider
Assignee: Tamas Szekeres

---

When loading the "Add MS Tables(s)" Dialog and connecting to a MSSQL database, determining the geometry type is very slow for large table, even if the "Use estimated table parameters" option is set for the connection.

Looking at https://github.com/qgis/Quantum-GIS/blob/d212ca7f7174a2063f245e31e27f2848387c1df0/src/providers/mssql/qgsmssqlsourceselect.cpp#L740 is seems like it should be working. Maybe the parameter is not being passed down correctly?

[qgib]

Author Name: Tamas Szekeres (@szekerest)

---

Fixed in git@github.com:szekerest/Quantum-GIS.git

[qgib]

Author Name: Tamas Szekeres (@szekerest)

---

Fixed in changeset "51011559c4d7cdb5a11e50cae23a20b688df2b77".

---

• status_id was changed from Open to Closed

[qgib]

Author Name: Jeremy Palmer (@palmerj)

---

Thanks. Can this stuff be backported to 1.8?

---

• assigned_to_id was changed from Tamas Szekeres to Jürgen Fischer

[qgib]

Author Name: Jeremy Palmer (@palmerj)

---

Hi Tamas,

I just re-tested this and it still seems that the estimated metadata parameter is not being used correctly. If the un-tick the "Only look in the geometry_columns metadata table" option large layers take a long time to update in the grid. Also when I add the layer to the canvas it takes a long time too (as it again tries to determine the table geometry type). Once the layer is added to the canvas, if I view the datasource URI in the metadata tab with the layer properties it doesn't have the estimated metadata parameter listed.

I'm using revision e54e87f trunk on winxp

Cheers
Jeremy

---

• status_id was changed from Closed to Reopened
• assigned_to_id was changed from Jürgen Fischer to Tamas Szekeres

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

• fixed_version_id was changed from Version 1.8.0 to Version 2.0.0

[qgib]

Author Name: Tamas Szekeres (@szekerest)

---

Fixed in changeset "51011559c4d7cdb5a11e50cae23a20b688df2b77".

---

• status_id was changed from Reopened to Closed