Error contact reading

[Albert Vaca Cintora]

It looks like in getColumnsFromContactsForIDs [1] we are passing to
many IDs. A solution probably would be we could request them in
smaller batches.

https://invent.kde.org/kde/kdeconnect-android/blob/master/src/org/kde/kdeconnect/Plugins/ContactsPlugin/ContactsPlugin.java#L161


On Sat, Oct 12, 2019 at 7:19 PM Simon Redman <simon at ergotech.com> wrote:
>
> Hmm. I have no idea what is causing this (since it's below Android's
> "we're totally not using a sqlite database" layer)
>
> Is this reliably reproducible? (If so, how?)
>
> On 10/12/19 8:48 AM, Aleix Pol wrote:
> > Hey Simon,
> > I was looking at adb logs, I saw this error, maybe you can take a look.
> >
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:
> > android.database.sqlite.SQLiteException: too many SQL variables (code
> > 1): , while compiling: SELECT lookup, contact_last_updated_timestamp
> > FROM view_contacts WHERE ((1)) AND (lookup
> > IN(?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,
> > ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?))
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > android.database.DatabaseUtils.readExceptionFromParcel(DatabaseUtils.java:179)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > android.database.DatabaseUtils.readExceptionFromParcel(DatabaseUtils.java:135)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > android.content.ContentProviderProxy.query(ContentProviderNative.java:418)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > android.content.ContentResolver.query(ContentResolver.java:754)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > android.content.ContentResolver.query(ContentResolver.java:704)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > android.content.ContentResolver.query(ContentResolver.java:662)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Helpers.ContactsHelper.getColumnsFromContactsForIDs(ContactsHelper.java:307)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Plugins.ContactsPlugin.ContactsPlugin.handleRequestAllUIDsTimestamps(ContactsPlugin.java:196)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Plugins.ContactsPlugin.ContactsPlugin.onPacketReceived(ContactsPlugin.java:253)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Device.onPacketReceived(Device.java:569)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Backends.BaseLink.packageReceived(BaseLink.java:84)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Backends.LanBackend.LanLink.receivedNetworkPacket(LanLink.java:255)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Backends.LanBackend.LanLink.lambda$reset$0$LanLink(LanLink.java:109)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > org.kde.kdeconnect.Backends.LanBackend.-$$Lambda$LanLink$TabvaCXA5qL_bcJDmIELWusNThw.run(Unknown
> > Source:4)
> > 10-12 17:36:27.053 29958  2983 E KDE/Device:    at
> > java.lang.Thread.run(Thread.java:764)
>