alibaba/GraphScope

Remaining improvement of headers cleanup for analytical engine

Open

Aperta il 17 mag 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (2 assegnatari)HTML (2401 star) (301 fork)batch import
component:gaeenhancementgood first issue

Descrizione

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

Guida contributor

Tech stack
cpp
Dominio
backendperformancebuild system
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 days
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
mostly clear
Prerequisiti
C++ knowledgeBuild system understandingFamiliarity with graph analytics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30
Direzione di ricerca
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.