Replies: 2 comments
-
The impact of this change is huge, it affects:
Is it still worth it? See branch |
Beta Was this translation helpful? Give feedback.
0 replies
-
My conclusion is to use IRI objects for entity IDs and to merge the |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
At present, this HQDM implementation uses
String
as the ID type which means that the RDF code has to convertStrings
to and fromIRI
objects, which not only looks messy but means a lot of new objects get created when they might not be needed.It is possible to parameterise the core types along the lines of
Thing<I>
whereI
is the type to use forThing
IDs, but:model
andimpl
packages.This would probably take 2 to 3 days and would be a very large pull request for someone to review, then all clients would need to change their code because they currently expect
String
IDs and would now haveIRI
IDs instead.I might be able to get this done as part of one of my projects, so I'll make a start and see what the impact is. Feel free to add comments, especially if you see any problems or a better way to do this.
Beta Was this translation helpful? Give feedback.
All reactions