nolanlawson/cordova-plugin-sqlite-2

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

Open

#46 aberto em 18 de dez. de 2016

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (29 forks)github user discovery
enhancementhelp wanted

Métricas do repositório

Stars
 (168 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Examine o arquivo SQLitePlugin.java, especificamente o método execute, e identifique onde a exceção do limite do CursorWindow é engolida. Implemente um bloco try catch ao redor da consulta ao banco de dados para capturar a exceção e lançar uma mensagem de erro significativa.
Pilha de tecnologia
javascriptjava
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitAndroid SDK
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70