Messaging cleanup

* Messaging cleanup

- Moved a couple internal log messages to FINE instead of INFO
- Cleaned up messaging when can't connect to server

Author: connelly38

CHANGELOG.md

@@ -2,6 +2,12 @@
 All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/).
 
+## Unreleased
+
+### Changed
+- Moved a couple internal log messages to FINE instead of INFO
+- Cleaned up messaging when can't connect to server
+
 ## [5.4.11] 2023-06-06
 
 ### Fixed

driver/src/main/java/oracle/nosql/driver/http/Client.java

@@ -855,9 +855,8 @@ public Result execute(Request kvRequest) {
                 }
                 throw e;
             } catch (IOException ioe) {
-                /* Maybe make this logFine */
                 String name = ioe.getClass().getName();
-                logInfo(logger, "Client execution IOException, name: " +
+                logFine(logger, "Client execution IOException, name: " +
                         name + ", message: " + ioe.getMessage());
                 /*
                  * An exception in the channel, e.g. the server may have

driver/src/main/java/oracle/nosql/driver/httpclient/HttpClient.java

@@ -382,9 +382,7 @@ public Channel getChannel(int timeoutMs)
             /* retry loop with at most (retryInterval) ms timeouts */
             long thisTimeoutMs = (timeoutMs - msDiff);
             if (thisTimeoutMs <= 0) {
-                String msg = "Timed out after " + msDiff +
-                             "ms (" + retries + " retries) trying " +
-                             "to acquire channel";
+                String msg = "Timed out trying to acquire channel";
                 logInfo(logger, "HttpClient " + name + " " + msg);
                 throw new TimeoutException(msg);
             }
@@ -397,7 +395,7 @@ public Channel getChannel(int timeoutMs)
                 retChan = fut.get(thisTimeoutMs, TimeUnit.MILLISECONDS);
             } catch (TimeoutException e) {
                 if (retries == 0) {
-                    logInfo(logger, "Timed out after " +
+                    logFine(logger, "Timed out after " +
                             (System.currentTimeMillis() - startMs) +
                             "ms trying to acquire channel: retrying");
                 }

driver/src/main/java/oracle/nosql/driver/package-info.java

@@ -107,7 +107,7 @@
  * both the SDK and the underlying Netty networking at level FINE. It will
  * log to both the console and a local file called "driver.log."
  * <pre>
-handlers=java.util.logging.FileHandler, java.util.logging.ConsoleHandler
+handlers=java.util.logging.FileHandler,java.util.logging.ConsoleHandler
 
 # File config
 java.util.logging.FileHandler.level=ALL
@@ -143,19 +143,19 @@
  *
  * Collection of stats are controlled by the following system
  * properties:<ol><li>
- *   -Dcom.oracle.nosql.sdk.nosqldriver.stats.profile=[none|regular|more|all]
- *   Specifies the stats profile: <i>none</i> - disabled,
- *      <i>regular</i> - per request: counters, errors, latencies, delays, retries
- *      <i>more</i> - stats above with 95th and 99th percentile latencies
- *      <i>all</i> - stats above with per query information.</li><li>
+ *   {@code -Dcom.oracle.nosql.sdk.nosqldriver.stats.profile=[none|regular|more|all]}<br>
+ *   Specifies the stats profile:<br> <i>none</i> - disabled<br>
+ *      <i>regular</i> - per request: counters, errors, latencies, delays, retries<br>
+ *      <i>more</i> - stats above with 95th and 99th percentile latencies<br>
+ *      <i>all</i> - stats above with per query information</li><li>
  *
- *   -Dcom.oracle.nosql.sdk.nosqldriver.stats.interval=600 Interval in
- *   seconds to log the stats, by default is 10 minutes.</li><li>
+ *   {@code -Dcom.oracle.nosql.sdk.nosqldriver.stats.interval=600}<br> Interval in
+ *   seconds to log the stats, default value is 600 seconds (10 minutes)</li><li>
  *
- *   -Dcom.oracle.nosql.sdk.nosqldriver.stats.pretty-print=true Option
+ *   {@code -Dcom.oracle.nosql.sdk.nosqldriver.stats.pretty-print=true}<br> Option
  *   to enable pretty printing of the JSON data, default value is false</li><li>
  *
- *   -Dcom.oracle.nosql.sdk.nosqldriver.stats.enable-log=false Option
+ *   {@code -Dcom.oracle.nosql.sdk.nosqldriver.stats.enable-log=false}<br> Option
  *   to turn on logging automatically if stats are enabled, default value is
  *   true</li>
  *   </ol><p>