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

RDF persistence to Neo4j graph database #1079

Merged
merged 29 commits into from
Oct 20, 2023
Merged

RDF persistence to Neo4j graph database #1079

merged 29 commits into from
Oct 20, 2023

Conversation

augustas
Copy link
Contributor

@augustas augustas commented Oct 20, 2023 •
edited
Loading

Intermediate PRs reviewed and tested previously:

Changes:

  • feat: poc attempt for Neo4j
  • feat: resource implementation language handling
  • fix: language value and relationship detection
  • Merge remote-tracking branch 'origin/develop' into poc/REL-1106/rdf-i…
  • test: make integrational tests running
  • feat: moved relation checking to property method for better reusability.
  • feat: changed visibility and type hints for inheritance
  • feat: fixed issue with UpdatedAt as a whole number
  • feat: introduced abstract filter class which different persistence im…
  • feat: system search implementation in neo4j and test coverage.
  • wip: extra changes required to run tests, but not production ready.
  • chore: cleanup imports and test to be more verbose.
  • Merge branch 'feature/REL-1193/system-search-implementation-in-neo4j'…
  • feat: rewrite complex search using neo4j.
  • feat: rewrite complex search using neo4j.
  • chore: provide PHP 7.4 compatibility.
  • chore: fix code styles and typos.
  • feature: made search persistance configurable.
  • fix: property/relationship identification
  • feature / REL-1193 / System search implementation in neo4j (feature / REL-1193 / System search implementation in neo4j #1056)
  • feature / REL-1199 / Complex Search (feature / REL-1199 / Complex Search #1060)
  • chore: removed questionable changes from changeset.
  • feat: getRdfTriples attempt
  • feat: ontology import/export for neo4j.
  • fix: removed duplicated properties.
  • fix: fixed tests and typo in functions.
  • Merge branch 'poc/REL-1106/rdf-in-neo4j' into fix/REL-1174/relationsh…
  • Merge pull request REL-1197: Rdf export/import from neo4j #1068 from oat-sa/feat/REL-1197/rdf-import-export-…

augustas and others added 28 commits May 24, 2023 15:30
@augustas
feat: poc attempt for Neo4j
b225703
@augustas
feat: resource implementation language handling
f6df1c1
@augustas
fix: language value and relationship detection
0966956
@vbyndych
Merge remote-tracking branch 'origin/develop' into poc/REL-1106/rdf-i…
a64fa30 
…n-neo4j
@vbyndych
test: make integrational tests running
bccb095 
- fixed constructor params as it caused exception in strict mode
- added return value as it is expected in order t oknow if anything was added or not
- re-wrote deprecated functionality in tests
- fixed tests as it was not in sync with logic.
- skipped those which require code update.
@vbyndych
feat: moved relation checking to property method for better reusability.
4de01d2
@vbyndych
feat: changed visibility and type hints for inheritance
4de1193
@vbyndych
feat: fixed issue with UpdatedAt as a whole number
11c3096
@vbyndych
feat: introduced abstract filter class which different persistence im…
74e2c7e 
…plementations can use.
@vbyndych
feat: system search implementation in neo4j and test coverage.
ce0b52e
@vbyndych
wip: extra changes required to run tests, but not production ready.
2d5b490
@vbyndych
chore: cleanup imports and test to be more verbose.
e7d06e3
@vbyndych
Merge branch 'feature/REL-1193/system-search-implementation-in-neo4j'…
87226ff 
… into feature/REL-1193/system-search-implementation-in-neo4j-extra
@vbyndych
feat: rewrite complex search using neo4j.
c9f5a30
@vbyndych
feat: rewrite complex search using neo4j.
4fa45b0
@vbyndych
chore: provide PHP 7.4 compatibility.
969248c
@vbyndych
chore: fix code styles and typos.
1b95fb6
@vbyndych
feature: made search persistance configurable.
56a8480
@augustas
fix: property/relationship identification
aaaaf26
@vbyndych
feature / REL-1193 / System search implementation in neo4j (#1056)
a88d8f4
@vbyndych
feature / REL-1199 / Complex Search (#1060)
27fa974
@vbyndych
chore: removed questionable changes from changeset.
01b806a
@augustas
feat: getRdfTriples attempt
b522bb1
@vbyndych
feat: ontology import/export for neo4j.
f081159
@vbyndych
fix: removed duplicated properties.
4828881
@vbyndych
fix: fixed tests and typo in functions.
0df3a8c
@augustas
Merge branch 'poc/REL-1106/rdf-in-neo4j' into fix/REL-1174/relationsh…
1e41b46 
…ip-handling
@augustas
Merge pull request #1068 from oat-sa/feat/REL-1197/rdf-import-export-…
6bdedfa 
…neo4j

REL-1197: Rdf export/import from neo4j
@augustas augustas changed the title Poc/rel 1106/rdf in neo4j RDF persistence to Neo4j graph database Oct 20, 2023
@vbyndych
Copy link
Contributor

@augustas Most if not all of the code styles issues failing here were fixed in eb99b90 does it make sense to extract it to even merge whole branch?

@github-actions
Copy link

Version

❕ Some commits are not using the conventional commits formats. They will be ignored in version management.

Target Version 15.30.0
Last version 15.29.0

There are 0 BREAKING CHANGE, 12 features, 4 fixes

@augustas augustas merged commit bd40001 into develop Oct 20, 2023
4 checks passed
@augustas augustas deleted the poc/REL-1106/rdf-in-neo4j branch October 20, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vbyndych vbyndych vbyndych approved these changes

@yaraslau-kavaliou yaraslau-kavaliou Awaiting requested review from yaraslau-kavaliou

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
@augustas @vbyndych