Skip to content

Fix several minor RDF4J Rio incompatibilities#548

Merged
Ostrzyciel merged 3 commits intomainfrom
piotr/fix-rdf4j-writer-compat
Sep 27, 2025
Merged

Fix several minor RDF4J Rio incompatibilities#548
Ostrzyciel merged 3 commits intomainfrom
piotr/fix-rdf4j-writer-compat

Conversation

@Ostrzyciel
Copy link
Member

Even more fixes to make eclipse-rdf4j/rdf4j#5449 possible:

  • Enable namespace declarations by default, as all RDF4J writers do that.
  • In case the RDF handler in the parser is unset, don't crash, just send the data to space.
  • Invoke clear() after and before parsing to make sure the base machinery can reset e.g., blank node hashes.

Copy link
Collaborator

@Karolina-Bogacka Karolina-Bogacka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes make sense

@Ostrzyciel Ostrzyciel enabled auto-merge (squash) September 27, 2025 14:28
@Ostrzyciel Ostrzyciel merged commit 180dfc1 into main Sep 27, 2025
11 checks passed
@Ostrzyciel Ostrzyciel deleted the piotr/fix-rdf4j-writer-compat branch September 27, 2025 14:31
@codecov
Copy link

codecov bot commented Sep 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ Complexity Δ
...everblink/jelly/convert/rdf4j/rio/JellyParser.java 87.80% <100.00%> (+0.62%) 0.00 <0.00> (ø)
...everblink/jelly/convert/rdf4j/rio/JellyWriter.java 78.88% <100.00%> (+12.22%) 0.00 <0.00> (ø)
...k/jelly/convert/rdf4j/rio/JellyWriterSettings.java 53.12% <100.00%> (ø) 0.00 <0.00> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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