afs commented on code in PR #2461: URL: https://github.com/apache/jena/pull/2461#discussion_r1615184599
########## jena-ontapi/src/main/resources/META-INF/services/org.apache.jena.sys.JenaSubsystemLifecycle: ########## @@ -0,0 +1,3 @@ +org.apache.jena.riot.system.InitRIOT +org.apache.jena.sparql.system.InitARQ +org.apache.jena.rdfs.sys.InitRDFS Review Comment: These should not be here. They are in other ServiceLoader files and wil be run from there. That might be the initialization problem. Putting them here makes them get called twice. Here, give a class in this module: ``` org.apache.jena.ontapi.sys.InitOntAPI ``` then new package and class: org.apache.jena.ontapi.sys.InitOntAPI ``` public class InitOntAPI implements JenaSubsystemLifecycle { @Override public void start() { // Any initialization for this module. This may be nothing. } @Override public void stop() {} @Override public int level() { return 15 ; -- this is between jena-core and jena-arq RIOT. } } ``` -- 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: pr-unsubscr...@jena.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@jena.apache.org For additional commands, e-mail: pr-h...@jena.apache.org