-
Notifications
You must be signed in to change notification settings - Fork 30
Feature/neptune #348
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
Merged
Merged
Feature/neptune #348
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…oudFormation template (#344) * Initial plan * Add Neptune plugin IAM authentication support with database driver Co-authored-by: jpmccu <[email protected]> * Address code review comments: add logging, imports, and documentation Co-authored-by: jpmccu <[email protected]> * Add Neptune IAM authentication documentation to ReadTheDocs Co-authored-by: jpmccu <[email protected]> * Expand Neptune how-to guide with detailed endpoint configuration Co-authored-by: jpmccu <[email protected]> * Fix Neptune entity resolver to correctly use FTS service with proper escaping Co-authored-by: jpmccu <[email protected]> * Add CloudFormation template for Neptune serverless setup with FTS Co-authored-by: jpmccu <[email protected]> * Refactor Neptune driver to use custom session instead of subclassing Co-authored-by: jpmccu <[email protected]> * Upgrade to rdflib 7.1.1 and simplify Neptune driver with custom session Co-authored-by: jpmccu <[email protected]> * Use aws_requests_auth and custom_requests parameter for simpler Neptune driver Co-authored-by: jpmccu <[email protected]> * Address code review: fix SPARQL injection, reuse session, improve efficiency Co-authored-by: jpmccu <[email protected]> * Add test for Neptune driver registration in plugin init Co-authored-by: jpmccu <[email protected]> * Remove redundant conditional check in driver registration test Co-authored-by: jpmccu <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: jpmccu <[email protected]>
) * Initial plan * Add temporary UUID graph functionality for Neptune GSP operations Co-authored-by: jpmccu <[email protected]> * Complete Neptune GSP temporary UUID graph implementation with tests Co-authored-by: jpmccu <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: jpmccu <[email protected]>
…to feature/neptune
…and improved error reporting (#347) * Initial plan * Add NeptuneBoto3Store with instance metadata support Co-authored-by: jpmccu <[email protected]> * Add comprehensive documentation for NeptuneBoto3Store Co-authored-by: jpmccu <[email protected]> * Add example script for NeptuneBoto3Store usage * Override query() and update() methods to enable authenticated access Co-authored-by: jpmccu <[email protected]> * Update neptune_driver to use NeptuneBoto3Store Co-authored-by: jpmccu <[email protected]> * Add raw_sparql_request interface and improve Neptune error reporting Co-authored-by: jpmccu <[email protected]> * debugging and sorting out the driver initialization issues * Fix query() and update() method signatures to properly override SPARQLStore interface Co-authored-by: jpmccu <[email protected]> * Fix _request method error handling and add comprehensive tests for response_mime_types usage Co-authored-by: jpmccu <[email protected]> * Fix HTTP error handling and add response_mime_types method to NeptuneBoto3Store Co-authored-by: jpmccu <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: jpmccu <[email protected]> Co-authored-by: Jamie McCusker <[email protected]>
|
✅ Vue.js tests passed! Coverage report available in artifacts. |
|
✅ Python tests completed! Coverage reports available in artifacts. |
|
✅ Vue.js tests passed! Coverage report available in artifacts. |
|
✅ Python tests completed! Coverage reports available in artifacts. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.