Skip to content

Fix rdf validate command in AoT builds#211

Merged
Ostrzyciel merged 2 commits intomainfrom
piotr/hotfix-validate
Aug 28, 2025
Merged

Fix rdf validate command in AoT builds#211
Ostrzyciel merged 2 commits intomainfrom
piotr/hotfix-validate

Conversation

@Ostrzyciel
Copy link
Member

We substituted the whole class instead of a single method...

I've found a better way around this, removing SecureRandom entirely. This reduces binary size from 65.11 MB to 63.96 MB. Incidental win, yay.

We substituted the whole class instead of a single method...

I've found a better way around this, removing SecureRandom entirely. This reduces binary size from 65.11 MB to 63.96 MB. Incidental win, yay.
@Ostrzyciel
Copy link
Member Author

Original bug report by @adanilenka

Commands like jelly-cli rdf validate ./to_jelly_quads_rdf_star_pos_002.jelly --compare-to-rdf-file=in_000.nq --compare-frame-indices=0 compare-ordered=true --debug fail for input files containing blank nodes.

Trace:

image

@Ostrzyciel Ostrzyciel merged commit 6ce169e into main Aug 28, 2025
7 checks passed
@Ostrzyciel Ostrzyciel deleted the piotr/hotfix-validate branch August 28, 2025 10:54
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