alibaba/GraphScope

[BUG] g.graphlearn()/g.gremlin() should check if the type is compatible first

Open

#1 509 ouverte le 6 mai 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)HTML (2 401 stars) (301 forks)batch import
component:pythonenhancementgood first issue

Description

Describe the bug

GraphLearn has implies int64_t as OID_T (see types.h) and current gremlin engine doesn't support string oids as well (see https://github.com/alibaba/GraphScope/issues/1265).

An exceptions should be thrown in g.graphlearn()/g.gremlin() is the oid type of vineyard graph is not compatible with those two engines.

Guide contributeur

Stack technique
cpp
Domaine
backend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic C++understand OID typesfamiliarity with GraphScope codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
Investigate the g.graphlearn() and g.gremlin() functions in the GraphScope codebase to identify where to add the type compatibility check. Refer to the types.h file (graphlearn/core/graph/storage/types.h) which defines OID T as int64 t. The exception should be thrown when the vineyard graph's OID type is not compatible. Also consider checking the linked issue #1265 for context on gremlin engine limitations. The fix should be localized and throw an appropriate error message.