This is an automated email from the ASF dual-hosted git repository.

mawiesne pushed a commit to branch migrate-tagging-server-to-opennlp-tools-2_1_0
in repository https://gitbox.apache.org/repos/asf/opennlp-sandbox.git

commit 9f8b1051c3bade460e9e9dbd26e7189d3ce49716
Author: Martin Wiesner <[email protected]>
AuthorDate: Fri Jan 20 15:10:18 2023 +0100

    updates sandbox component 'tagging-server' to be compatible with latest 
opennlp-tools release
    
    - adjusts opennlp-tools to 2.1.0
    - adjusts parent project (org.apache.apache) to version 18
    - adjusts Java language level to 11
    - updates `maven-bundle-plugin` to version 5.1.8 so things work with class 
files compiled in Java 11 format
    - adjusts some array declarations to comply with Java, not C style
    - removes unused imports
---
 tagging-server/pom.xml                             | 171 ++++++++++-----------
 .../opennlp/tagging_server/POSTaggerResource.java  |   4 +-
 .../apache/opennlp/tagging_server/ServiceUtil.java |   6 +-
 .../tagging_server/TaggingServerApplication.java   |   2 +-
 .../tagging_server/TaggingServerBundle.java        |   4 +-
 .../namefind/BratNameFinderResource.java           |  55 ++++---
 .../namefind/DefaultRawTextNameFinderFactory.java  |   6 +-
 .../namefind/NameFinderResource.java               |  33 ++--
 .../namefind/RawTextNameFinderFactory.java         |   3 +
 .../tagging_server/postag/POSTaggerResource.java   |   8 +-
 .../sentdetect/SentenceDetectorResource.java       |   6 +-
 .../tagging_server/tokenize/TokenizerResource.java |  10 +-
 12 files changed, 148 insertions(+), 160 deletions(-)

diff --git a/tagging-server/pom.xml b/tagging-server/pom.xml
index 93b1c09..4d41e0d 100644
--- a/tagging-server/pom.xml
+++ b/tagging-server/pom.xml
@@ -21,59 +21,50 @@
 
 <project xmlns="http://maven.apache.org/POM/4.0.0"; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; 
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/maven-v4_0_0.xsd";>
        <modelVersion>4.0.0</modelVersion>
-
        <parent>
                <groupId>org.apache</groupId>
                <artifactId>apache</artifactId>
-               <version>9</version>
+               <!-- TODO OPENNLP-1452 once this is resolved, move to 29 as 
well. -->
+               <version>18</version>
                <relativePath />
        </parent>
 
        <groupId>org.apache.opennlp</groupId>
        <artifactId>tagging-server</artifactId>
-       <version>0.0.1-SNAPSHOT</version>
+       <version>2.1.1-SNAPSHOT</version>
        <packaging>bundle</packaging>
 
-       <name>OpenNLP Tagging Server</name>
+       <name>Apache OpenNLP Tagging Server</name>
 
        <prerequisites>
                <maven>3.0</maven>
        </prerequisites>
-
-       <repositories>
-               <repository>
-                   <id>maven2-repository.java.net</id>
-                   <name>Java.net Repository for Maven</name>
-                   <url>http://download.java.net/maven/2/</url>
-                   <layout>default</layout>
-               </repository>
-       </repositories>
        
        <dependencies>
        
                <dependency>
                  <groupId>org.apache.opennlp</groupId>
                  <artifactId>opennlp-tools</artifactId>
-                 <version>1.5.2-incubating</version>
+                 <version>2.1.0</version>
                </dependency>
                
-           <dependency>
-               <groupId>javax.servlet</groupId>
-               <artifactId>servlet-api</artifactId>
-               <version>2.5</version>
-       </dependency>
-    
-        <dependency>
-            <groupId>org.osgi</groupId>
-            <artifactId>org.osgi.core</artifactId>
-            <version>4.2.0</version>
-        </dependency>
-        
-        <dependency>
-            <groupId>org.osgi</groupId>
-            <artifactId>org.osgi.compendium</artifactId>
-            <version>4.2.0</version>
-        </dependency>
+               <dependency>
+                               <groupId>javax.servlet</groupId>
+                               <artifactId>servlet-api</artifactId>
+                               <version>2.5</version>
+               </dependency>
+
+               <dependency>
+                               <groupId>org.osgi</groupId>
+                               <artifactId>org.osgi.core</artifactId>
+                               <version>4.2.0</version>
+               </dependency>
+
+               <dependency>
+                               <groupId>org.osgi</groupId>
+                               <artifactId>org.osgi.compendium</artifactId>
+                               <version>4.2.0</version>
+               </dependency>
 
                <dependency>
                        <groupId>com.sun.jersey</groupId>
@@ -97,7 +88,7 @@
                <dependency>
                        <groupId>junit</groupId>
                        <artifactId>junit</artifactId>
-                       <version>4.8.1</version>
+                       <version>4.13.2</version>
                        <scope>test</scope>
                </dependency>
        </dependencies>
@@ -108,66 +99,68 @@
                                <groupId>org.apache.maven.plugins</groupId>
                                <artifactId>maven-compiler-plugin</artifactId>
                                <configuration>
-                                       <source>1.6</source>
-                                       <target>1.6</target>
-                               <compilerArgument>-Xlint</compilerArgument>
+                                       <source>11</source>
+                                       <target>11</target>
+                                       
<compilerArgument>-Xlint</compilerArgument>
                                </configuration>
                        </plugin>
                        
                        <!-- to generate the MANIFEST-FILE required by the 
bundle -->
-            <plugin>
-                <groupId>org.apache.felix</groupId>
-                <artifactId>maven-bundle-plugin</artifactId>
-                <version>2.3.7</version>
-                <extensions>true</extensions>
-                <executions>
-                    <execution>
-                        <id>bundle-manifest</id>
-                        <phase>process-classes</phase>
-                        <goals>
-                            <goal>manifest</goal>
-                        </goals>
-                    </execution>
-                </executions>
-                <configuration>
-                    
<manifestLocation>${project.build.directory}/META-INF</manifestLocation>
-                    <instructions>
-                        
<Bundle-SymbolicName>${project.artifactId}</Bundle-SymbolicName>
-                        <Import-Package>
-                               !javax.persistence.*,
-                               !javax.servlet.annotation,
-                               !javax.servlet.jsp.*,
-                               !javax.microedition.*,
-                               !javax.mail.*,
-                               !javax.transaction.xa,
-                               !javax.jms,
-                               !javax.interceptor,!javax.inject,
-                               !javax.enterprise.*,
-                               !javax.annotation.security,
-                               !javax.ejb,
-                               !com.sun.xml.fastinfoset.*,
-                               !org.jvnet.*,
-                               !org.apache.derby.impl.drda,
-                               !com.ibm.jvm,
-                               !com.sun.jdmk.comm,
-                               !com.sun.net.httpserver,
-                               !sun.misc,
-                               javax.servlet,
-                               *,
-                               com.sun.jersey.json.impl.provider.entity.*,
-                               com.sun.jersey.json.*,
-                               org.codehaus.jackson.*,
-                               org.codehaus.jackson.xc,
-                               com.sun.jersey.api.core,
-                               
com.sun.jersey.spi.container.servlet</Import-Package>
-                        <!-- 
Import-Package>!com.ibm.jvm,!com.sun.jdmk.comm,!com.sun.net.httpserver,*,com.sun.jersey.api.core,com.sun.jersey.spi.container.servlet</Import-Package-->
-                        
<Export-Package>org.apache.opennlp.tagging_server.*</Export-Package>
-                        <Webapp-Context>rest</Webapp-Context>
-                        <Web-ContextPath>rest</Web-ContextPath>
-                        
<Bundle-Activator>org.apache.opennlp.tagging_server.TaggingServerBundle</Bundle-Activator>
-                    </instructions>
-                </configuration>
-            </plugin>
-             </plugins>
+                       <plugin>
+                                       <groupId>org.apache.felix</groupId>
+                                       
<artifactId>maven-bundle-plugin</artifactId>
+                                       <version>5.1.8</version>
+                                       <extensions>true</extensions>
+                                       <executions>
+                                                       <execution>
+                                                                       
<id>bundle-manifest</id>
+                                                                       
<phase>process-classes</phase>
+                                                                       <goals>
+                                                                               
        <goal>manifest</goal>
+                                                                       </goals>
+                                                       </execution>
+                                       </executions>
+                                       <configuration>
+                                                       
<manifestLocation>${project.build.directory}/META-INF</manifestLocation>
+                                                       <instructions>
+                                                                       
<Bundle-SymbolicName>${project.artifactId}</Bundle-SymbolicName>
+                                                                       
<Import-Package>
+                                                                               
!javax.persistence.*,
+                                                                               
!javax.servlet.annotation,
+                                                                               
!javax.servlet.jsp.*,
+                                                                               
!javax.microedition.*,
+                                                                               
!javax.mail.*,
+                                                                               
!javax.transaction.xa,
+                                                                               
!javax.jms,
+                                                                               
!javax.interceptor,
+                                                                               
!javax.inject,
+                                                                               
!javax.enterprise.*,
+                                                                               
!javax.annotation.security,
+                                                                               
!javax.ejb,
+                                                                               
!com.sun.xml.fastinfoset.*,
+                                                                               
!org.jvnet.*,
+                                                                               
!org.apache.derby.impl.drda,
+                                                                               
!com.ibm.jvm,
+                                                                               
!com.sun.jdmk.comm,
+                                                                               
!com.sun.net.httpserver,
+                                                                               
!sun.misc,
+                                                                               
javax.servlet,
+                                                                               
*,
+                                                                               
com.sun.jersey.json.impl.provider.entity.*,
+                                                                               
com.sun.jersey.json.*,
+                                                                               
org.codehaus.jackson.*,
+                                                                               
org.codehaus.jackson.xc,
+                                                                               
com.sun.jersey.api.core,
+                                                                               
com.sun.jersey.spi.container.servlet
+                                                                       
</Import-Package>
+                                                                       <!-- 
Import-Package>!com.ibm.jvm,!com.sun.jdmk.comm,!com.sun.net.httpserver,*,com.sun.jersey.api.core,com.sun.jersey.spi.container.servlet</Import-Package-->
+                                                                       
<Export-Package>org.apache.opennlp.tagging_server.*</Export-Package>
+                                                                       
<Webapp-Context>rest</Webapp-Context>
+                                                                       
<Web-ContextPath>rest</Web-ContextPath>
+                                                                       
<Bundle-Activator>org.apache.opennlp.tagging_server.TaggingServerBundle</Bundle-Activator>
+                                               </instructions>
+                               </configuration>
+                       </plugin>
+               </plugins>
        </build>
 </project>
\ No newline at end of file
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/POSTaggerResource.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/POSTaggerResource.java
index a0b59ac..cec06f7 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/POSTaggerResource.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/POSTaggerResource.java
@@ -40,7 +40,7 @@ public class POSTaggerResource {
   // @QueryParam("model") String modelName
   public String[][] tag(String document[][]) {
     
-    ServiceReference modelService = ModelUtil.getModelService(POSModel.class);
+    ServiceReference modelService = ModelUtil.getService(POSModel.class);
     
     try {
       String[][] tags = new String[document.length][];
@@ -53,7 +53,7 @@ public class POSTaggerResource {
       return tags;
     }
     finally {
-      ModelUtil.releaseModel(modelService);
+      ModelUtil.releaseService(modelService);
     }
   }
 }
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/ServiceUtil.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/ServiceUtil.java
index a4e6340..90b314c 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/ServiceUtil.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/ServiceUtil.java
@@ -27,9 +27,6 @@ public class ServiceUtil {
 
   private ServiceUtil() {
   }
-  
-  
-  
 
   public static ServiceReference getModelServiceReference(
       Class<?> serviceClazz, String modelName) {
@@ -41,8 +38,7 @@ public class ServiceUtil {
 
     ServiceReference[] serviceReferences;
     try {
-      serviceReferences = context.getServiceReferences(
-          null, filter);
+      serviceReferences = context.getServiceReferences(null, filter);
     } catch (InvalidSyntaxException e) {
       throw new IllegalArgumentException("modelName can't be used as value in 
filter!", e);
     }
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerApplication.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerApplication.java
index 7de2984..4981d7f 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerApplication.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerApplication.java
@@ -30,7 +30,7 @@ public class TaggingServerApplication extends Application {
   
   @Override
   public Set<Class<?>> getClasses() {
-    Set<Class<?>> result = new HashSet<Class<?>>();
+    Set<Class<?>> result = new HashSet<>();
     result.add(POSTaggerResource.class);
     result.add(NameFinderResource.class);
     result.add(BratNameFinderResource.class);
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerBundle.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerBundle.java
index c05fdda..d231d7e 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerBundle.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/TaggingServerBundle.java
@@ -57,9 +57,7 @@ public class TaggingServerBundle implements BundleActivator {
         try {
           httpService.registerServlet("/rest", new ServletContainer(), 
jerseyServletParams, null);
           httpService.registerResources("/","/htmls",null);
-        } catch (ServletException e) {
-          throw new RuntimeException(e);
-        } catch (NamespaceException e) {
+        } catch (ServletException | NamespaceException e) {
           throw new RuntimeException(e);
         }
         
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/BratNameFinderResource.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/BratNameFinderResource.java
index 7402ec9..059229a 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/BratNameFinderResource.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/BratNameFinderResource.java
@@ -12,14 +12,14 @@ import javax.ws.rs.Produces;
 import javax.ws.rs.QueryParam;
 import javax.ws.rs.core.MediaType;
 
+import org.apache.opennlp.tagging_server.ServiceUtil;
+import org.osgi.framework.ServiceReference;
+
 import opennlp.tools.namefind.TokenNameFinder;
 import opennlp.tools.sentdetect.SentenceDetector;
 import opennlp.tools.tokenize.Tokenizer;
 import opennlp.tools.util.Span;
 
-import org.apache.opennlp.tagging_server.ServiceUtil;
-import org.osgi.framework.ServiceReference;
-
 @Path("/bratner")
 public class BratNameFinderResource {
 
@@ -55,35 +55,35 @@ public class BratNameFinderResource {
 
       SentenceDetector sentDetect = nameFinderFactory.createSentenceDetector();
       Tokenizer tokenizer = nameFinderFactory.createTokenizer();
-      TokenNameFinder nameFinders[] = nameFinderFactory.createNameFinders();
+      TokenNameFinder[] nameFinders = nameFinderFactory.createNameFinders();
 
-      Span sentenceSpans[] = sentDetect.sentPosDetect(text);
+      Span[] sentenceSpans = sentDetect.sentPosDetect(text);
 
-      Map<String, NameAnn> map = new HashMap<String, NameAnn>();
+      Map<String, NameAnn> map = new HashMap<>();
 
       int indexCounter = 0;
 
-      for (int i = 0; i < sentenceSpans.length; i++) {
+      for (Span sentenceSpan : sentenceSpans) {
         // offset of sentence gets lost here!
-        Span tokenSpans[] = tokenizer.tokenizePos(sentenceSpans[i]
-            .getCoveredText(text).toString());
+        Span[] tokenSpans = tokenizer.tokenizePos(sentenceSpan
+                .getCoveredText(text).toString());
+
+        String[] tokens = Span.spansToStrings(tokenSpans, text);
 
-        String tokens[] = Span.spansToStrings(tokenSpans, text);
-        
         for (TokenNameFinder nameFinder : nameFinders) {
-          Span names[] = nameFinder.find(tokens);
+          Span[] names = nameFinder.find(tokens);
 
           for (Span name : names) {
             int beginOffset = tokenSpans[name.getStart()].getStart()
-                + sentenceSpans[i].getStart();
+                    + sentenceSpan.getStart();
             int endOffset = tokenSpans[name.getEnd() - 1].getEnd()
-                + sentenceSpans[i].getStart();
+                    + sentenceSpan.getStart();
 
             // create a list of new line indexes
             List<Integer> newLineIndexes = new ArrayList<Integer>();
-            
+
             // TODO: Code needs to handle case that there are multiple new 
lines in a row
-            
+
             boolean inNewLineSequence = false;
             for (int ci = beginOffset; ci < endOffset; ci++) {
               if (text.charAt(ci) == '\n' || text.charAt(ci) == '\r') {
@@ -91,40 +91,39 @@ public class BratNameFinderResource {
                   newLineIndexes.add(ci);
                 }
                 inNewLineSequence = true;
-              }
-              else {
+              } else {
                 inNewLineSequence = false;
               }
             }
-            
-            List<String> textSegments = new ArrayList<String>();
-            List<int[]> spanSegments = new ArrayList<int[]>();
-            
+
+            List<String> textSegments = new ArrayList<>();
+            List<int[]> spanSegments = new ArrayList<>();
+
             int segmentBegin = beginOffset;
-            
+
             for (int newLineOffset : newLineIndexes) {
               // create segment from begin to offset
               textSegments.add(text.substring(segmentBegin, newLineOffset));
               spanSegments.add(new int[]{segmentBegin, newLineOffset});
-              
+
               segmentBegin = findNextNonWhitespaceChar(text, newLineOffset + 
1, endOffset);
-              
+
               if (segmentBegin == -1) {
                 break;
               }
             }
-            
+
             // create left over segment
             if (segmentBegin != -1) {
               textSegments.add(text.substring(segmentBegin, endOffset));
               spanSegments.add(new int[]{segmentBegin, endOffset});
             }
-            
+
             NameAnn ann = new NameAnn();
             ann.texts = textSegments.toArray(new String[textSegments.size()]);
             ann.offsets = spanSegments.toArray(new int[spanSegments.size()][]);
             ann.type = name.getType();
-            
+
             map.put(Integer.toString(indexCounter++), ann);
           }
         }
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/DefaultRawTextNameFinderFactory.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/DefaultRawTextNameFinderFactory.java
index 7510e05..f3dc970 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/DefaultRawTextNameFinderFactory.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/DefaultRawTextNameFinderFactory.java
@@ -31,12 +31,12 @@ public class DefaultRawTextNameFinderFactory implements 
RawTextNameFinderFactory
   
   private final SentenceModel sentModel;
   private final TokenizerModel tokenModel;
-  private final TokenNameFinderModel nameModels[];
+  private final TokenNameFinderModel[] nameModels;
 
   // TODO: How can this be an array of models with blueprint?!
   
   public DefaultRawTextNameFinderFactory(SentenceModel sentModel,
-      TokenizerModel tokenModel, TokenNameFinderModel nameModels[]) {
+      TokenizerModel tokenModel, TokenNameFinderModel[] nameModels) {
     this.sentModel = sentModel;
     this.tokenModel = tokenModel;
     this.nameModels = nameModels;
@@ -55,7 +55,7 @@ public class DefaultRawTextNameFinderFactory implements 
RawTextNameFinderFactory
   @Override
   public TokenNameFinder[] createNameFinders() {
     
-    TokenNameFinder nameFinders[] = new TokenNameFinder[nameModels.length];
+    TokenNameFinder[] nameFinders = new TokenNameFinder[nameModels.length];
     
     for (int i = 0; i < nameFinders.length; i++) {
       nameFinders[i] = new NameFinderME(nameModels[i]);
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/NameFinderResource.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/NameFinderResource.java
index a2fc20d..5c95657 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/NameFinderResource.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/NameFinderResource.java
@@ -26,6 +26,9 @@ import javax.ws.rs.Path;
 import javax.ws.rs.Produces;
 import javax.ws.rs.core.MediaType;
 
+import org.apache.opennlp.tagging_server.ServiceUtil;
+import org.osgi.framework.ServiceReference;
+
 import opennlp.tools.namefind.NameFinderME;
 import opennlp.tools.namefind.TokenNameFinder;
 import opennlp.tools.namefind.TokenNameFinderModel;
@@ -33,16 +36,12 @@ import opennlp.tools.sentdetect.SentenceDetector;
 import opennlp.tools.tokenize.Tokenizer;
 import opennlp.tools.util.Span;
 
-import org.apache.opennlp.tagging_server.ServiceUtil;
-import org.osgi.framework.ServiceReference;
-
-
 @Path("/namefinder")
 public class NameFinderResource {
 
   public static class NameFinderDocument {
-    private List<Span[]> document;
-    private List<Span[]> names;
+    private final List<Span[]> document;
+    private final List<Span[]> names;
     
     NameFinderDocument(List<Span[]> document, List<Span[]> names) {
       this.document = document;
@@ -58,11 +57,11 @@ public class NameFinderResource {
     }
   }
   
-  private List<Span[]> find(TokenNameFinder nameFinders[], String[][] 
document) {
+  private List<Span[]> find(TokenNameFinder[] nameFinders, String[][] 
document) {
 
-    List<Span[]> names = new ArrayList<Span[]>();
+    List<Span[]> names = new ArrayList<>();
 
-    for (String sentence[] : document) {
+    for (String[] sentence : document) {
       for (TokenNameFinder nameFinder : nameFinders) {
         names.add(nameFinder.find(sentence));
       }
@@ -83,9 +82,9 @@ public class NameFinderResource {
       NameFinderME nameFinder = new NameFinderME(
           ServiceUtil.getService(modelService, TokenNameFinderModel.class));
       
-      List<Span[]> names = new ArrayList<Span[]>();
+      List<Span[]> names = new ArrayList<>();
       
-      for (String sentence[] : document) {
+      for (String[] sentence : document) {
         names.add(nameFinder.find(sentence));
       }
       
@@ -114,20 +113,20 @@ public class NameFinderResource {
       SentenceDetector sentDetect = factory.createSentenceDetector();
       Tokenizer tokenizer = factory.createTokenizer();
       
-      Span sentenceSpans[] = sentDetect.sentPosDetect(document);
+      Span[] sentenceSpans = sentDetect.sentPosDetect(document);
       
-      List<Span[]> tokenizedSentencesSpan = new ArrayList<Span[]>();
+      List<Span[]> tokenizedSentencesSpan = new ArrayList<>();
       String[][] tokenizedSentences = new String[sentenceSpans.length][];
       
       for (int i = 0; i < sentenceSpans.length; i++) {
         // offset of sentence gets lost here!
-        Span tokenSpans[] = 
tokenizer.tokenizePos(sentenceSpans[i].getCoveredText(document).toString());
+        Span[] tokenSpans = 
tokenizer.tokenizePos(sentenceSpans[i].getCoveredText(document).toString());
         // all spans need to be sentence offset adjusted!
         tokenSpans = offsetSpans(tokenSpans, sentenceSpans[i].getStart());
         
         tokenizedSentencesSpan.add(tokenSpans);
         
-        String tokens[] = new String[tokenSpans.length];
+        String[] tokens = new String[tokenSpans.length];
         for (int ti = 0; ti < tokenSpans.length; ti++) {
           tokens[ti] = tokenSpans[ti].getCoveredText(document).toString();
         }
@@ -135,7 +134,7 @@ public class NameFinderResource {
         tokenizedSentences[i] = tokens;
       }
       
-      TokenNameFinder nameFinders[] = factory.createNameFinders();
+      TokenNameFinder[] nameFinders = factory.createNameFinders();
       
       return new NameFinderDocument(tokenizedSentencesSpan, find(nameFinders, 
tokenizedSentences));
     }
@@ -147,7 +146,7 @@ public class NameFinderResource {
   private Span[] offsetSpans(
       Span[] tokenSpans, int offset) {
     
-    Span spans[] = new Span[tokenSpans.length];
+    Span[] spans = new Span[tokenSpans.length];
     
     for (int i = 0; i < tokenSpans.length; i++) {
       spans[i] = new Span(tokenSpans[i].getStart() + offset,
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/RawTextNameFinderFactory.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/RawTextNameFinderFactory.java
index 711deb5..add90d1 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/RawTextNameFinderFactory.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/namefind/RawTextNameFinderFactory.java
@@ -27,6 +27,9 @@ import opennlp.tools.tokenize.Tokenizer;
  */
 public interface RawTextNameFinderFactory {
   SentenceDetector createSentenceDetector();
+
   Tokenizer createTokenizer();
+
   TokenNameFinder[] createNameFinders();
+  
 }
\ No newline at end of file
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/postag/POSTaggerResource.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/postag/POSTaggerResource.java
index c0a3ad7..c453fcb 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/postag/POSTaggerResource.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/postag/POSTaggerResource.java
@@ -23,13 +23,13 @@ import javax.ws.rs.Path;
 import javax.ws.rs.Produces;
 import javax.ws.rs.core.MediaType;
 
+import org.apache.opennlp.tagging_server.ServiceUtil;
+import org.osgi.framework.ServiceReference;
+
 import opennlp.tools.postag.POSModel;
 import opennlp.tools.postag.POSTagger;
 import opennlp.tools.postag.POSTaggerME;
 
-import org.apache.opennlp.tagging_server.ServiceUtil;
-import org.osgi.framework.ServiceReference;
-
 @Path("/postagger")
 public class POSTaggerResource {
 
@@ -39,7 +39,7 @@ public class POSTaggerResource {
   @Path("_tag")
   // @QueryParam("lang") String lang,
   // @QueryParam("model") String modelName
-  public String[][] tag(String document[][]) {
+  public String[][] tag(String[][] document) {
     
     ServiceReference modelService = 
ServiceUtil.getServiceReference(POSModel.class);
     
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/sentdetect/SentenceDetectorResource.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/sentdetect/SentenceDetectorResource.java
index 75cdbee..26e4165 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/sentdetect/SentenceDetectorResource.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/sentdetect/SentenceDetectorResource.java
@@ -26,14 +26,14 @@ import javax.ws.rs.Path;
 import javax.ws.rs.Produces;
 import javax.ws.rs.core.MediaType;
 
+import org.apache.opennlp.tagging_server.ServiceUtil;
+import org.osgi.framework.ServiceReference;
+
 import opennlp.tools.sentdetect.SentenceDetector;
 import opennlp.tools.sentdetect.SentenceDetectorME;
 import opennlp.tools.sentdetect.SentenceModel;
 import opennlp.tools.util.Span;
 
-import org.apache.opennlp.tagging_server.ServiceUtil;
-import org.osgi.framework.ServiceReference;
-
 @Path("/sentdetect")
 public class SentenceDetectorResource {
 
diff --git 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/tokenize/TokenizerResource.java
 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/tokenize/TokenizerResource.java
index 3a62a5f..3501d14 100644
--- 
a/tagging-server/src/main/java/org/apache/opennlp/tagging_server/tokenize/TokenizerResource.java
+++ 
b/tagging-server/src/main/java/org/apache/opennlp/tagging_server/tokenize/TokenizerResource.java
@@ -26,13 +26,13 @@ import javax.ws.rs.Path;
 import javax.ws.rs.Produces;
 import javax.ws.rs.core.MediaType;
 
+import org.apache.opennlp.tagging_server.ServiceUtil;
+import org.osgi.framework.ServiceReference;
+
 import opennlp.tools.tokenize.TokenizerME;
 import opennlp.tools.tokenize.TokenizerModel;
 import opennlp.tools.util.Span;
 
-import org.apache.opennlp.tagging_server.ServiceUtil;
-import org.osgi.framework.ServiceReference;
-
 @Path("/tokenize")
 public class TokenizerResource {
 
@@ -47,7 +47,7 @@ public class TokenizerResource {
       TokenizerME tokenizer = new TokenizerME(
           ServiceUtil.getService(modelService, TokenizerModel.class));
       
-      List<String[]> tokenizedSentences = new ArrayList<String[]>();
+      List<String[]> tokenizedSentences = new ArrayList<>();
       
       for (String sentence : document) {
         tokenizedSentences.add(tokenizer.tokenize(sentence));
@@ -71,7 +71,7 @@ public class TokenizerResource {
       TokenizerME tokenizer = new TokenizerME(
               ServiceUtil.getService(modelService, TokenizerModel.class));
 
-      List<Span[]> tokenizedSentences = new ArrayList<Span[]>();
+      List<Span[]> tokenizedSentences = new ArrayList<>();
       
       for (String sentence : document) {
         tokenizedSentences.add(tokenizer.tokenizePos(sentence));

Reply via email to