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

Reply via email to