[ 
https://issues.apache.org/jira/browse/OPENNLP-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680934#comment-17680934
 ] 

ASF GitHub Bot commented on OPENNLP-1448:
-----------------------------------------

rzo1 commented on code in PR #492:
URL: https://github.com/apache/opennlp/pull/492#discussion_r1087617892


##########
opennlp-dl/src/test/java/opennlp/dl/doccat/DocumentCategorizerDLEval.java:
##########
@@ -29,13 +29,17 @@
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import opennlp.dl.AbstactDLTest;
 import opennlp.dl.InferenceOptions;
 import opennlp.dl.doccat.scoring.AverageClassificationScoringStrategy;
 
 public class DocumentCategorizerDLEval extends AbstactDLTest {
 
+  private static Logger logger = 
LoggerFactory.getLogger(DocumentCategorizerDLEval.class);

Review Comment:
   missing `final`



##########
opennlp-dl/src/main/java/opennlp/dl/doccat/DocumentCategorizerDL.java:
##########
@@ -51,6 +53,7 @@
  */
 public class DocumentCategorizerDL implements DocumentCategorizer {
 
+  private Logger logger = LoggerFactory.getLogger(DocumentCategorizerDL.class);

Review Comment:
   missing `static final`





> Introduce SLF4J in OpenNLP
> --------------------------
>
>                 Key: OPENNLP-1448
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1448
>             Project: OpenNLP
>          Issue Type: Sub-task
>            Reporter: Richard Zowalla
>            Assignee: Richard Zowalla
>            Priority: Major
>
> This will be the first step regarding OPENNLP-1447.
> Goal is to replace System.err / System.out calls with logger output, which is 
> configurable.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to