mozilla/mentat

Extend query builder to allow composing parts of the query

Open

#649 aberto em 20 de abr. de 2018

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Rust (116 forks)github user discovery
A-queryenhancementgood first bughelp wanted

Métricas do repositório

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

Description

Right now, the query builder takes a blob of EDN and allows to bind values. I would like to be able to compose the parts of the query in a granular fashion. By manipulating the string, I hacked in support for :order and :limit, but I'd like to be able to say

let qb = QueryBuilder::new();
qb.add_find("?x ?y");
qb.add_where("[?x :is 10]");
if limit {
  qb.set_limit("20");
}
if sort {
  qb.set_order("?y ?x");
}

I think that parsing any string given to the QueryBuilder into a FindQuery up front, and then updating the FindQuery with the builder pattern, should do the trick.

This is follow-up to https://github.com/mozilla/mentat/pull/602.

Guia do colaborador

Direção de pesquisa
Entenda os tipos existentes QueryBuilder e FindQuery e implemente um padrão builder para permitir a composição de partes como add find, add where, set limit, set order.
Pilha de tecnologia
rust
Domain
database
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65