Revert logging changes

Author: abhishekrb19

server/src/main/java/org/apache/druid/query/lookup/LookupModule.java

@@ -32,7 +32,6 @@
 import org.apache.druid.guice.LazySingleton;
 import org.apache.druid.guice.LifecycleModule;
 import org.apache.druid.initialization.DruidModule;
-import org.apache.druid.java.util.common.logger.Logger;
 import org.apache.druid.query.dimension.LookupDimensionSpec;
 import org.apache.druid.query.expression.LookupExprMacro;
 import org.apache.druid.server.initialization.jetty.JettyBindings;
@@ -43,15 +42,13 @@
 
 public class LookupModule implements DruidModule
 {
-  private static final Logger log = new Logger(LookupModule.class);
   static final String PROPERTY_BASE = "druid.lookup";
   public static final String FAILED_UPDATES_KEY = "failedUpdates";
   public static final int LOOKUP_LISTENER_QOS_MAX_REQUESTS = 2;
 
   @Override
   public List<? extends Module> getJacksonModules()
   {
-    log.info("Umm yes LookupModule.getJacksonModules()");
     return ImmutableList.<Module>of(
         new SimpleModule("DruidLookupModule").registerSubtypes(MapLookupExtractorFactory.class),
         new SimpleModule().registerSubtypes(
@@ -64,7 +61,6 @@ public List<? extends Module> getJacksonModules()
   @Override
   public void configure(Binder binder)
   {
-    log.info("Umm yes LookupModule.configure()");
     JsonConfigProvider.bind(binder, PROPERTY_BASE, LookupConfig.class);
     binder.bind(LookupExtractorFactoryContainerProvider.class).to(LookupReferencesManager.class);
     LifecycleModule.register(binder, LookupReferencesManager.class);
@@ -83,7 +79,6 @@ public void configure(Binder binder)
   @LazySingleton
   public LookupNodeService getLookupNodeService(LookupListeningAnnouncerConfig lookupListeningAnnouncerConfig)
   {
-    log.info("Umm yes LookupModule.getLookupNodeService()");
     return new LookupNodeService(lookupListeningAnnouncerConfig.getLookupTier());
   }
 }

server/src/main/java/org/apache/druid/query/lookup/LookupReferencesManager.java

@@ -118,7 +118,6 @@ public LookupReferencesManager(
   )
   {
     this(lookupConfig, objectMapper, coordinatorClient, lookupListeningAnnouncerConfig, false);
-    LOG.info("Umm Initializing LookupReferencesManager");
   }
 
   @VisibleForTesting

services/src/main/java/org/apache/druid/cli/CliPeon.java

@@ -325,7 +325,6 @@ public Task readTask(@Json ObjectMapper mapper, ExecutorLifecycleConfig config,
                 // write the remote task.json to the task file location for ExecutorLifecycle to pickup
                 FileUtils.write(config.getTaskFile(), task, Charset.defaultCharset());
               }
-              log.info("Grrr read task from config.getTaskFile() [%s]", config.getTaskFile());
               return mapper.readValue(config.getTaskFile(), Task.class);
             }
             catch (IOException e) {

services/src/main/java/org/apache/druid/cli/CliPeonLoadSpecHolder.java

@@ -23,7 +23,6 @@
 import com.google.inject.Provider;
 import org.apache.druid.error.DruidException;
 import org.apache.druid.indexing.common.task.Task;
-import org.apache.druid.java.util.common.logger.Logger;
 import org.apache.druid.server.coordination.BroadcastDatasourceLoadingSpec;
 import org.apache.druid.server.lookup.cache.LookupLoadingSpec;
 import org.apache.druid.server.metrics.LoadSpecHolder;
@@ -36,16 +35,13 @@
  */
 public class CliPeonLoadSpecHolder implements LoadSpecHolder
 {
-  private static final Logger log = new Logger(CliPeonLoadSpecHolder.class);
-
   private Provider<Task> taskProvider;
 
   @Inject
   public CliPeonLoadSpecHolder(
       Provider<Task> taskProvider
   )
   {
-    log.info("GRRRONCE NewTaskLoadSpecHolder with [%s]", taskProvider);
     this.taskProvider = taskProvider;
   }
 
@@ -54,7 +50,6 @@ public CliPeonLoadSpecHolder(
   public LookupLoadingSpec getLookupLoadingSpec()
   {
     final Task task = taskProvider.get();
-    log.info("GRRR NewTaskLoadSpecHolder.getLookupLoadingSpec() task[%s]", task);
     if (task == null) {
       throw DruidException.defensive("blah");
     }
@@ -65,7 +60,6 @@ public LookupLoadingSpec getLookupLoadingSpec()
   public BroadcastDatasourceLoadingSpec getBroadcastDatasourceLoadingSpec()
   {
     final Task task = taskProvider.get();
-    log.info("GRRR NewTaskLoadSpecHolder.getBroadcastDatasourceLoadingSpec() task[%s]", task);
     if (task == null) {
       throw DruidException.defensive("blah");
     }

services/src/main/java/org/apache/druid/cli/CliPeonTaskHolder.java

@@ -23,7 +23,6 @@
 import com.google.inject.Provider;
 import org.apache.druid.error.DruidException;
 import org.apache.druid.indexing.common.task.Task;
-import org.apache.druid.java.util.common.logger.Logger;
 import org.apache.druid.server.metrics.TaskHolder;
 
 /**
@@ -34,23 +33,20 @@
  */
 public class CliPeonTaskHolder implements TaskHolder
 {
-  private static final Logger log = new Logger(CliPeonTaskHolder.class);
   private Provider<Task> taskProvider;
 
   @Inject
   public CliPeonTaskHolder(
       Provider<Task> taskProvider
   )
   {
-    log.info("GRRRONCE NewTaskPropertiesHolder with [%s]", taskProvider);
     this.taskProvider = taskProvider;
   }
 
   @Override
   public String getDataSource()
   {
     final Task task = taskProvider.get();
-    log.info("GRRR NewTaskPropertiesHolder.getDataSource() task[%s]", task);
     if (task == null) {
       throw DruidException.defensive("blah");
     }
@@ -61,7 +57,6 @@ public String getDataSource()
   public String getTaskId()
   {
     final Task task = taskProvider.get();
-    log.info("GRRR NewTaskPropertiesHolder.getDataSource() task[%s]", task);
     if (task == null) {
       throw DruidException.defensive("blah");
     }