afs commented on PR #3347: URL: https://github.com/apache/jena/pull/3347#issuecomment-3148410472
I took a look at removing jena-core org.apache.jena.ontology. There are two references outside jena-core (`schemagen` in jena-cmds, and some old code in TDB1 for old non-transactional uses cases). That's all good. Inside `jena-core` the situation is not too bad but not trivial. A few factories have ontology operations. Mostly, they need to be deprecated and point to `jena-ontapi`. The two big areas are the assembler subsystem and the reasoner support for owl reasoning. These look more complicated to disentangle. It is not that the reasoner code uses org.apache.jena.ontology (mostly, it doesn't in 'main', some in 'test') but it needs the OWL vocabulary file). So I'm happy to merge this PR - including the noted caveat about copied code. OK? Doing changes in steps is more manageable; and byJena6 we will definitely be in a better place even if the time needed to do the whole job doesn't line up. Even then, we can signal the intent and not be morally bound to a "Jena7" for it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
