Skip to content

Jena/RDF4J: increase default size of serializer lookups#550

Merged
Ostrzyciel merged 6 commits intomainfrom
piotr/rdf4j-default-writer-settings
Sep 28, 2025
Merged

Jena/RDF4J: increase default size of serializer lookups#550
Ostrzyciel merged 6 commits intomainfrom
piotr/rdf4j-default-writer-settings

Conversation

@Ostrzyciel
Copy link
Member

We used to default to the very small name/prefix tables that reduce performance drastically when dealing with larger datasets. Meanwhile, in small datasets these larger lookups perform the same or just slightly worse than the small ones. It makes more sense to use the larger ones as the default.

We used to default to the very small name/prefix tables that reduce performance drastically when dealing with larger datasets. Meanwhile, in small datasets these larger lookups perform the same or just slightly worse than the small ones. It makes more sense to use the larger ones as the default.
@codecov
Copy link

codecov bot commented Sep 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ Complexity Δ
...nk/jelly/convert/jena/riot/JellyFormatVariant.java 88.63% <100.00%> (ø) 0.00 <0.00> (ø)
...erblink/jelly/convert/jena/riot/JellyLanguage.java 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
...k/jelly/convert/rdf4j/rio/JellyWriterSettings.java 53.12% <100.00%> (ø) 0.00 <0.00> (ø)
...k/jelly/convert/titanium/TitaniumJellyEncoder.java 100.00% <100.00%> (ø) 2.00 <1.00> (ø)
...nk/jelly/convert/titanium/TitaniumJellyWriter.java 100.00% <100.00%> (ø) 2.00 <1.00> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ostrzyciel Ostrzyciel enabled auto-merge (squash) September 28, 2025 17:47
@Ostrzyciel Ostrzyciel merged commit 74c67b1 into main Sep 28, 2025
12 checks passed
@Ostrzyciel Ostrzyciel deleted the piotr/rdf4j-default-writer-settings branch September 28, 2025 17:48
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