Queries fail to return if sub-relation table has real world amount of data

Abierto

#237 abierto el 11/7/2019 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 4/5
Tiempo estimado: 3-5 días
Aptitud para principiantes: 35/100

Tipo de issue: Error
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: graphql, python, sqlalchemy
Área: api, database, performance

Línea de trabajo

Comienza con graphene/types/resolver.py, especialmente con attr_resolver, y reproduce la consulta anidada tableA/tableB mostrada en el issue contra una tabla relacionada grande. Sigue cómo la resolución de TableA hace que se acceda a la relación TableB y, a continuación, verifica que la consulta devuelve el resultado en un tiempo razonable sin cargar toda la tabla relacionada.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

In a one to many if you have a table setup as follows (a very simple one to many relationship), the GraphQL query will timeout if tableB has any real world amount of data.

TableB has about 400k records in total. Graphene-sqlalchemy will query for TableA, and then SQLAlchemy itself will try resolve the relationship of the table when graphene calls for it. When this happens the query is sent to the DB but it won't return in enough time (its pulling back ~1GB of data as it tries to pull every single field on the sub table).

I've narrowed it down to graphene/types/resolver.py

def attr_resolver(attname, default_value, root, info, **args):
    return getattr(root, attname, default_value)

Graphene captures the TableA information, and then when TableB get resolved, the root parameter is populated with the "TableA" model, which SQLAlchemy then tries to completely resolve 400k records, query times out and then fails.

query{
  tableA(first: 10){
    edges{
      node{
        tableB(first: 5){
          edges{
            node{
              fieldA
            }
          }
        }
      }
    }
  }
}

Lenguaje dominante: Python
Estrellas: 985
Forks: 224
Métricas de merge de PR: Sin PR fusionados en 30 d