Error contact reading

[Simon Redman]

Yes, probably we could do this. The trouble is that for the first time running synchronization the desktop would then call this method with the complete list of changed contacts. Since it is first sync, it would be all of them, thus it would have the same problem.

So the simplest solution would be to just break it in to batches since this solves both cases. But what is a safe batch size?

On October 12, 2019 12:01:53 PM PDT, Albert Vaca Cintora <albertvaka at gmail.com> wrote:
>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)
>> > >
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20191013/f10c7fff/attachment.html>