anongit / mytracks

Automatically exported from code.google.com/p/mytracks
0 stars 0 forks source link

Properly set the projection argument of SQLiteQueryBuilder.htmlquery() #165

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
According to:

http://www.ideasandroid.com/android/sdk/docs/reference/android/database/sqlite/S
QLiteQueryBuilder.html#query(android.database.sqlite.SQLiteDatabase, 
java.lang.String[], java.lang.String, java.lang.String[], java.lang.String, 
java.lang.String, java.lang.String, java.lang.String)

projectionIn    A list of which columns to return. Passing null will return all 
columns, which is discouraged to prevent reading data from storage that isn't 
going to be used.

As far as I can tell we never set projectionIn.  This looks wasteful and it 
should be easy to fix.

Original issue reported on code.google.com by sandordo...@google.com on 28 Sep 2010 at 12:05

GoogleCodeExporter commented 9 years ago
We set projection in ONE case, which I think also needs fixing - 
getLastLocationId / getLastTrackId. Those are fake queries where the real query 
is a subquery. I'm not sure if we can put min(_id) in the projection, but it's 
possible, it should be done.

Original comment by rdama...@google.com on 28 Sep 2010 at 12:15

GoogleCodeExporter commented 9 years ago

Original comment by sandordo...@google.com on 18 Feb 2011 at 4:51

GoogleCodeExporter commented 9 years ago

Original comment by jshih@google.com on 23 Jun 2012 at 5:26

GoogleCodeExporter commented 9 years ago

Original comment by jshih@google.com on 17 Jul 2012 at 11:21