Error contact reading

[Albert Vaca Cintora]

Actually here is the call for all the IDs [1], the previous one only
requests a single ID.

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

On Sat, Oct 12, 2019 at 8:57 PM Albert Vaca Cintora
<albertvaka at gmail.com> wrote:
>
> 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)
> >