Fix logger/LOGGER confusion.

eamonnmcmanus · eamonnmcmanus · commit 86a615a566af · 2020-04-28T13:56:57.000-07:00
diff --git a/functions/ocr/ocr-translate-text/src/main/java/functions/OcrTranslateText.java b/functions/ocr/ocr-translate-text/src/main/java/functions/OcrTranslateText.java
@@ -36,7 +36,7 @@
 import java.util.logging.Logger;
 
 public class OcrTranslateText implements BackgroundFunction<PubSubMessage> {
-  private static final Logger LOGGER = Logger.getLogger(OcrTranslateText.class.getName());
+  private static final Logger logger = Logger.getLogger(OcrTranslateText.class.getName());
 
   // TODO<developer> set these environment variables
   private static final String PROJECT_ID = getenv("GCP_PROJECT");
@@ -56,7 +56,7 @@ public void accept(PubSubMessage pubSubMessage, Context context) {
         pubSubMessage.getData().getBytes(StandardCharsets.UTF_8));
 
     String targetLang = ocrMessage.getLang();
-    LOGGER.info("Translating text into " + targetLang);
+    logger.info("Translating text into " + targetLang);
 
     // Translate text to target language
     String text = ocrMessage.getText();
@@ -73,15 +73,15 @@ public void accept(PubSubMessage pubSubMessage, Context context) {
       response = client.translateText(request);
     } catch (IOException e) {
       // Log error (since IOException cannot be thrown by a function)
-      LOGGER.log(Level.SEVERE, "Error translating text: " + e.getMessage(), e);
+      logger.log(Level.SEVERE, "Error translating text: " + e.getMessage(), e);
       return;
     }
     if (response.getTranslationsCount() == 0) {
       return;
     }
 
     String translatedText = response.getTranslations(0).getTranslatedText();
-    LOGGER.info("Translated text: " + translatedText);
+    logger.info("Translated text: " + translatedText);
 
     // Send translated text to (subsequent) Pub/Sub topic
     String filename = ocrMessage.getFilename();
@@ -92,10 +92,10 @@ public void accept(PubSubMessage pubSubMessage, Context context) {
       PubsubMessage pubsubApiMessage = PubsubMessage.newBuilder().setData(byteStr).build();
 
       publisher.publish(pubsubApiMessage).get();
-      LOGGER.info("Text translated to " + targetLang);
+      logger.info("Text translated to " + targetLang);
     } catch (InterruptedException | ExecutionException e) {
       // Log error (since these exception types cannot be thrown by a function)
-      LOGGER.log(Level.SEVERE, "Error publishing translation save request: " + e.getMessage(), e);
+      logger.log(Level.SEVERE, "Error publishing translation save request: " + e.getMessage(), e);
     }
   }
 
