alibaba/GraphScope

Remaining improvement of headers cleanup for analytical engine

Open

#1,582 opened on May 17, 2022

View on GitHub
 (3 comments) (0 reactions) (2 assignees)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

Contributor guide

Tech stack
cpp
Domain
backendperformancebuild system
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
C++ knowledgeBuild system understandingFamiliarity with graph analytics
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30
Research direction
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.