Rework some log messages

Author: littleaj

core/src/main/java/com/microsoft/applicationinsights/internal/channel/samplingV2/FixedRateSamplingTelemetryProcessor.java

@@ -170,7 +170,7 @@ public boolean process(Telemetry telemetry) {
 
                 if (SamplingScoreGeneratorV2.getSamplingScore(telemetry) >= sp) {
 
-                    InternalLogger.INSTANCE.info("Item %s sampled out", telemetry.getClass().getSimpleName());
+                    InternalLogger.INSTANCE.trace("Item %s sampled out", telemetry.getClass().getSimpleName());
                     return false;
                 }
 

core/src/main/java/com/microsoft/applicationinsights/internal/heartbeat/HeartBeatModule.java

@@ -59,37 +59,38 @@ public HeartBeatModule(Map<String, String> properties) {
             try {
               setHeartBeatInterval(Long.parseLong(entry.getValue()));
             } catch (Exception e) {
-              InternalLogger.INSTANCE.trace("Exception while adding Heartbeat interval,"
-                  + " Stack trace is: %s", ExceptionUtils.getStackTrace(e));
+              if (InternalLogger.INSTANCE.isTraceEnabled()) {
+                InternalLogger.INSTANCE.trace("Exception while adding Heartbeat interval: %s", ExceptionUtils.getStackTrace(e));
+              }
             }
             break;
           case "isHeartBeatEnabled":
             try {
               setHeartBeatEnabled(Boolean.parseBoolean(entry.getValue()));
-            }
-            catch (Exception e) {
-              InternalLogger.INSTANCE.trace("Exception while adding enabling/disabling heartbeat,"
-                  + " Stack trace is: %s", ExceptionUtils.getStackTrace(e));
+            } catch (Exception e) {
+              if (InternalLogger.INSTANCE.isTraceEnabled()) {
+                InternalLogger.INSTANCE.trace("Exception while adding enabling/disabling heartbeat: %s", ExceptionUtils.getStackTrace(e));
+              }
             }
             break;
           case "ExcludedHeartBeatPropertiesProvider":
             try {
               List<String> excludedHeartBeatPropertiesProviderList = parseStringToList(entry.getValue());
               setExcludedHeartBeatPropertiesProvider(excludedHeartBeatPropertiesProviderList);
-            }
-            catch (Exception e) {
-              InternalLogger.INSTANCE.trace("Exception while adding Excluded Heartbeat providers,"
-                  + " Stack trace is: %s", ExceptionUtils.getStackTrace(e));
+            } catch (Exception e) {
+              if (InternalLogger.INSTANCE.isTraceEnabled()) {
+                InternalLogger.INSTANCE.trace("Exception while adding Excluded Heartbeat providers: %s", ExceptionUtils.getStackTrace(e));
+              }
             }
             break;
           case "ExcludedHeartBeatProperties":
             try {
               List<String> excludedHeartBeatPropertiesList = parseStringToList(entry.getValue());
               setExcludedHeartBeatProperties(excludedHeartBeatPropertiesList);
-            }
-            catch (Exception e) {
-              InternalLogger.INSTANCE.trace("Exception while adding excluded heartbeat properties,"
-                  + " Stack trace is: %s", ExceptionUtils.getStackTrace(e));
+            } catch (Exception e) {
+              if (InternalLogger.INSTANCE.isTraceEnabled()) {
+                InternalLogger.INSTANCE.trace("Exception while adding excluded heartbeat properties: %s", ExceptionUtils.getStackTrace(e));
+              }
             }
             break;
           default:

web/src/main/java/com/microsoft/applicationinsights/web/internal/httputils/HttpServerHandler.java

@@ -140,8 +140,9 @@ public void handleEnd(ServletRequest request, ServletResponse response,
      */
     public void handleException(Exception e) {
         try {
-            InternalLogger.INSTANCE.trace("Unhandled exception while processing request: %s",
-                ExceptionUtils.getStackTrace(e));
+            if (InternalLogger.INSTANCE.isTraceEnabled()) {
+                InternalLogger.INSTANCE.trace("Unhandled exception while processing request: %s", ExceptionUtils.getStackTrace(e));
+            }
             if (telemetryClient != null) {
                 telemetryClient.trackException(e);
             }