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

[#1220] Retain GraphHead in operators not changing the structure #1235

Merged
merged 16 commits into from May 7, 2019

Conversation

Projects
None yet
4 participants
@timo95
Copy link
Contributor

commented Apr 23, 2019

This includes the operators:

  • Exclusion
  • ReduceExclusion
  • Subgraph
  • HITS
  • LabelPropagation
  • SingleSourceShortestPaths
  • DistinctVertexDegrees

And Transformation functions:

  • ConnectNeighbors
  • EdgeToVertex
  • PropagatePropertyToNeighbor
  • VertexToEdge

ApplySubgraph will be done in #1233.

Fixes #1220.

timo95 added some commits Apr 17, 2019

ChrizZz110 and others added some commits Apr 24, 2019

[#1220] Use getFactory
Co-Authored-By: timo95 <24251362+timo95@users.noreply.github.com>
[#1220] Use getFactory
Co-Authored-By: timo95 <24251362+timo95@users.noreply.github.com>
[#1220] Use getFactory
Co-Authored-By: timo95 <24251362+timo95@users.noreply.github.com>
[#1220] Use getFactory
Co-Authored-By: timo95 <24251362+timo95@users.noreply.github.com>

@timo95 timo95 force-pushed the timo95:1220-graphhead branch from 869b561 to 8f46c4e Apr 24, 2019

@p-f

p-f approved these changes Apr 25, 2019

Copy link
Collaborator

left a comment

Looks good, just one thing (although that is technically not in the scope of this issue)

timo95 added some commits Apr 30, 2019

@ChrizZz110
Copy link
Contributor

left a comment

Good job. There are only two minor things regarding the javadoc. After that, we can merge.

@ChrizZz110 ChrizZz110 self-assigned this May 3, 2019

timo95 added some commits May 7, 2019

@galpha

galpha approved these changes May 7, 2019

Copy link
Contributor

left a comment

LGTM - good work

Comments got fixed

@galpha galpha merged commit 76ff80c into dbs-leipzig:develop May 7, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@timo95 timo95 deleted the timo95:1220-graphhead branch May 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.