alibaba/GraphScope

Remaining improvement of headers cleanup for analytical engine

Open

#1 582 ouverte le 17 mai 2022

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

Métriques du dépôt

Stars
 (2 401 stars)
Métriques de merge PR
 (Merge moyen 1m) (8 PRs mergées en 30 j)

Description

Is your feature request related to a problem? Please describe.

  • Clean up headers of frame/*.cc to reduce building time of project/building app
  • Split the project_frame.cc base on the projected fragment type, only include the related fragment header
  • Clean up headers of tests .cc like test/run_app.cc

Describe the solution you'd like A clear and concise description of what you want to happen.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context #1541

Guide contributeur

Direction de recherche
Examinez l'issue référencée #1541 pour le contexte sur le nettoyage précédent des en têtes. Consultez les fichiers frame/*.cc et les tests comme test/run app.cc pour comprendre la structure actuelle des inclusions. Envisagez de diviser project frame.cc par type de fragment projeté comme suggéré. Passez en revue les commentaires pour toute directive supplémentaire des mainteneurs.
Stack technique
c
Domaine
backend
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.
2
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60