Skip to content

Conversation

@kmcginnes
Copy link
Collaborator

@kmcginnes kmcginnes commented Oct 15, 2025

Description

  • Cleanup old sparql templates no longer used before the consolidation of mapping logic
  • Migrate remaining code to use SparqlValue instead of RawValue
  • Update vertex details query to use rdf type uri constant

Validation

  • Tested in Neptune

Related Issues

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0
    license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@kmcginnes kmcginnes changed the title Sparql/minor tweaks Final tweaks for consolidated mapping logic Oct 15, 2025
@kmcginnes kmcginnes force-pushed the sparql/minor-tweaks branch from 5ff63bc to ec1b6c5 Compare October 15, 2025 23:25
@kmcginnes kmcginnes force-pushed the sparql/minor-tweaks branch from ec1b6c5 to 83def52 Compare October 16, 2025 23:52
@kmcginnes kmcginnes marked this pull request as ready for review October 16, 2025 23:53
@kmcginnes kmcginnes force-pushed the sparql/minor-tweaks branch from 83def52 to 3f1b405 Compare October 16, 2025 23:54
@kmcginnes kmcginnes changed the title Final tweaks for consolidated mapping logic Minor tweaks for consolidated mapping logic Oct 16, 2025
@kmcginnes kmcginnes merged commit 2eb5b88 into aws:main Oct 20, 2025
2 checks passed
@kmcginnes kmcginnes deleted the sparql/minor-tweaks branch October 20, 2025 21:08
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.

2 participants