glom r1880 - trunk/glom/utility_widgets/db_adddel

[murrayc svn gnome org]

Author: murrayc
Date: Thu Feb  5 18:01:32 2009
New Revision: 1880
URL: http://svn.gnome.org/viewvc/glom?rev=1880&view=rev

Log:
Added a TODO_Performance hint for later.

Modified:
   trunk/glom/utility_widgets/db_adddel/glom_db_treemodel.cc

Modified: trunk/glom/utility_widgets/db_adddel/glom_db_treemodel.cc
==============================================================================
--- trunk/glom/utility_widgets/db_adddel/glom_db_treemodel.cc	(original)
+++ trunk/glom/utility_widgets/db_adddel/glom_db_treemodel.cc	Thu Feb  5 18:01:32 2009
@@ -489,9 +489,10 @@
       // does not support it (for example the one for Sqlite recordsets does
       // not). The alternative would be to acquire a random-access model
       // directly here for SQLite, but this would
-      // a) make this code dependent on the database backend used
+      // a) make this code dependent on the database backend used.
       // b) fetch rows we perhaps don't need, if only the first few rows of
       // a table are accessed.
+      // TODO_Performance: The unnecessary (for PostgreSQL) extra indirection might theoretically make this slower.
       m_gda_datamodel = Gnome::Gda::DataAccessWrapper::create(m_gda_datamodel);
 
       if(app && app->get_show_sql_debug())