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.

Sign up for GitHub

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

Jump to bottom

Graph Features for Prism Interface #519

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

Graph Features for Prism Interface #519

wants to merge 36 commits into from

Conversation

TobiasHafner
Copy link
Contributor

@TobiasHafner TobiasHafner commented Oct 9, 2024 •
edited
Loading

This pull request adds features related to the labeled property graph model to the prism interface. Most notably:

  • Executions of graph queries returning graph results (includes relational ones)
  • Addition of representations for graph elements such as nodes, edges and paths for result
  • Unit tests for different types of graph queries and their results
  • Code cleanup related to above changes (uniform names for variables and concepts, usage of new java language features)

@TobiasHafner TobiasHafner requested a review from gartens October 9, 2024 15:05
Tobias Hafner and others added 29 commits October 21, 2024 10:37
@gartens
Prepare statement processing for the introduction of graphs
93e474e
@gartens
Improve code readability
bb40400
@gartens
Add basic graph execution
cc9e6b6
@gartens
Add relational graph results
d92e9bc
@gartens
Add graph results
585dbd6
@gartens
Add graph serialization tests
eb63649
@gartens
Use mavenLocal for driver
e536fed
@gartens
Add testcase for relational multimodel operations
c44d234
@gartens
Add testcase for multi-model relational metadata
28989ed
@gartens
Add test for multimodel mql
e89ad2a
@gartens
Fix ddl result bug
8933073
@gartens
Rephrase error message
aad24ef
@gartens
Add cypher tests
5cc1df7
@gartens
Remove broken assertion
a660672
@gartens
Adjust test
c1642f8
@gartens
Fix node serialization
eb4b060
@gartens
Add not implemented exception and todo for paths
aa036e5
@gartens
Update api version
5091b41
@gartens
Put classes into corresponding packages
0a86c9d
@gartens
Fix method access
3bedb68
@gartens
Update to new Prism API files
208d7c3
@gartens
Use switch expression
7ce3517
@gartens
Rename source/target to left/right
c1635e6
@gartens
Fix fallout from rebase
7b00a38
@gartens
Fix instances missed in previous
038d96d
@gartens
Improve switch expression
5a65a20
@gartens
Use .toList() instead of Collectors.toList()
badae33
@gartens
Remove unused state and errorCode from PIServiceExceptions
c748e29
@gartens
Make getExecutor switch over the data models
45ba7a9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gartens gartens Awaiting requested review from gartens

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TobiasHafner @gartens