mozilla/mentat

Extend query builder to allow composing parts of the query

Open

#649 ouverte le 20 avr. 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Rust (116 forks)github user discovery
A-queryenhancementgood first bughelp wanted

Métriques du dépôt

Stars
 (1 661 stars)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Comprenez les types QueryBuilder et FindQuery existants, puis implémentez un modèle constructeur pour permettre la composition de parties comme add find, add where, set limit, set order.
Stack technique
rust
Domaine
database
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65