nolanlawson/cordova-plugin-sqlite-2

Android: should throw meaningful errors when the 1MB CursorWindow limit is reached

Open

#46 geöffnet am 18. Dez. 2016

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (29 Forks)github user discovery
enhancementhelp wanted

Repository-Metriken

Stars
 (168 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Currently these errors just get swallowed, e.g.: https://github.com/pouchdb/pouchdb/issues/5413

We just return empty results which causes bugs later on down the line that are hard to diagnose: https://github.com/nolanlawson/cordova-plugin-sqlite-2/blob/33afdef8409b8782b7a82ca7d59270e7e11541dd/src/android/SQLitePlugin.java#L93-L96

Contributor Guide

Research-Richtung
Untersuchen Sie die Datei SQLitePlugin.java, insbesondere die Methode execute, und identifizieren Sie, wo die CursorWindow Limit Ausnahme verschluckt wird. Implementieren Sie einen try catch Block um die Datenbankabfrage herum, um die Ausnahme abzufangen und eine aussagekräftige Fehlermeldung zu werfen.
Tech Stack
javascriptjava
Domain
mobile
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitAndroid SDK
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70