Error contact reading

[Albert Vaca Cintora]

Or even, if we are requesting *all* the IDs, I guess we could remove
the `WHERE ID IN()` in that call?

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