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 (2 401 stars) (301 forks)batch import
component:gaeenhancementgood first issue

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

Stack technique
cpp
Domaine
backendperformancebuild system
Type d'issue
feature
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.
1-2 days
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.
mostly clear
Prérequis
C++ knowledgeBuild system understandingFamiliarity with graph analytics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Examine the referenced issue #1541 for context on prior header cleanup. Look at files frame/*.cc and tests like test/run app.cc to understand current include structure. Consider splitting project frame.cc by projected fragment type as suggested. Review comments for any additional maintainer guidance.