alibaba/GraphScope

Remaining improvement of headers cleanup for analytical engine

Open

#1.582 geöffnet am 17. Mai 2022

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)HTML (2.401 Stars) (301 Forks)batch import
component:gaeenhancementgood first issue

Beschreibung

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

Contributor Guide

Tech Stack
cpp
Domain
backendperformancebuild system
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
C++ knowledgeBuild system understandingFamiliarity with graph analytics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
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.