Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introducing the Graph module #22

Open
wants to merge 221 commits into
base: master
Choose a base branch
from
Open

Introducing the Graph module #22

wants to merge 221 commits into from

Conversation

FlorentinD
Copy link

Result of two consecutive projects (first one by @TimPF).

  • 2 Graph Storage Formats (CSR and AdjacencyGraph), which use MorphStore columns for compression
  • Graph Algorithms: Breadth-First Search and Page-Rank
  • a couple of "micro-benchmarks" (maybe not really micro)
  • Introduces a morph-flag for the path to resource directory of an LDBC-Graph (needed for tests and benchmark)
  • a morph operator which saves the block-offsets
  • a template free column-base class (currently used by the graph formats)

Also lifts the needed C++ version to 17.

It might make sense to introduce an graph build flag.

pfeiffer-tim and others added 30 commits April 30, 2019 11:25
Florentin Doerre added 28 commits June 2, 2020 23:15
finding a current bug (fixed in the following)
.. added the offset to early
actually need to decompress the blocks one by one .. (see `morph_saving_offsets_test` for reasoning)

Lead to a couple of changes:
* morph_saving_offsets now accepts column_with_offsets as input
* dedicated morph_saving_offset for decompression
* changing cache in csr -> not const function anymore
catching a current bug regarding the offset_column (not fixing it yet)

Failure case: last vertex has no edges -> no edges added -> no offset set
slimming down graph.h 
unique ptr didn't make sense, as the algorithm does not own the graph here
for evaluating impact of graph compressions
and correcting the number of block-offsets to be reserved
looks like the ldbc graph schema is not final .. they added a forum type in the dev branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants