sharpetronics/QGIS
1
0
Fork 0
mirror of https://github.com/qgis/QGIS.git synced 2025-02-25 00:58:06 -05:00
Code Issues Packages Projects Releases Wiki Activity

Decrease time taken to find key column in layers that come from postgres

Browse Source 
views


git-svn-id: http://svn.osgeo.org/qgis/trunk@6685 c8812cc2-4d05-0410-92ff-de0c093fc19c
This commit is contained in:
g_j_m 2007-02-24 02:05:41 +00:00
parent eae25d870f
commit a4a7d7c04f
1 changed files with 22 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
src/providers/postgres/qgspostgresprovider.cpp
View File

@ -1251,21 +1251,31 @@ void QgsPostgresProvider::findColumns(tableCols& cols)
if (!viewDef.isEmpty())
{
// This regular expression needs more testing. Since the view
// definition comes from postgresql and has been 'standardised', we
// don't need to deal with everything that the user could put in a view
// definition. Does the regexp have to deal with the schema??
// Compiling and executing the regexp for each row from the above query
// can take quite a while - a database can easily have hundreds of
// rows. Working on the premise that we are only doing this to catch the
// cases where the view column has been renamed using the AS construct,
// we'll check for that first before doing the potentially
// time-consuming regular expression.
QRegExp s(".* \"?" + QRegExp::escape(temp.table_name) +
"\"?\\.\"?" + QRegExp::escape(temp.column_name) +
"\"? AS \"?(\\w+)\"?,* .*");
if (viewDef.contains("AS"))
{
// This regular expression needs more testing. Since the view
// definition comes from postgresql and has been 'standardised', we
// don't need to deal with everything that the user could put in a view
// definition. Does the regexp have to deal with the schema??
QRegExp s(".* \"?" + QRegExp::escape(temp.table_name) +
"\"?\\.\"?" + QRegExp::escape(temp.column_name) +
"\"? AS \"?(\\w+)\"?,* .*");
QgsDebugMsg(viewDef + "\n" + s.pattern());
QgsDebugMsg(viewDef + "\n" + s.pattern());
if (s.indexIn(viewDef) != -1)
{
temp.view_column_name = s.cap(1);
}
if (s.indexIn(viewDef) != -1)
{
temp.view_column_name = s.cap(1);
}
}
}
QgsDebugMsg(temp.view_schema + "."