probcomp/bayeslite

pass query parameters through to metamodel syntax

Open

#446 aberto em 12 de jul. de 2016

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (63 forks)batch import
help wantedk days

Métricas do repositório

Stars
 (915 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Currently we do not distinguish

CREATE GENERATOR g FOR p USING m ("foo")

from

CREATE GENERATOR g FOR p USING m ('foo')

which is not at the moment blocking anything (hence the stall of #410), but which may be worth fixing soon. Furthermore, passing a verbatim string through, e.g. to provide the source code in

CREATE GENERATOR g FOR p USING cgpm (
    MODEL x GIVEN y USING venturescript (source = 'assume x = 42; ...')
)

as we kludgily do in https://github.com/probcomp/bayeslite/blob/f3f27a584286a4f3f3a415a7935da6ea78bd9824/tests/test_vscgpm.py#L166-L179 with Python string formatting, is not easy to do reliably. That fragment just assumes no " appears in the VentureScript code, and abuses "..." quotation, which in SQL means names, with '...' meaning strings. No quotation should be necessary on the user's part -- it should be possible to do

bdb.execute('CREATE GENERATOR ... USING venturescript (source = ?) ...', (venturescript,))

and get the same intended effect reliably.

Guia do colaborador

Direção de pesquisa
Investigue como os parâmetros SQL (ex.: '?') são atualmente tratados no analisador SQL e garanta que sejam passados corretamente para a sintaxe do metamodelo, evitando o abuso de literais de string. Olhe para o arquivo de gramática e o manipulador para instruções CREATE GENERATOR.
Pilha de tecnologia
python
Domain
backenddatabase
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
SQLPython
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75