From 93322bb8ef0bfdefe1b83644c4be4e518528491c Mon Sep 17 00:00:00 2001
From: Zhen <zhen.li@neotechnology.com>
Date: Thu, 4 May 2017 16:11:37 +0200
Subject: [PATCH] Added some missing API docs

---
 .../java/org/neo4j/driver/v1/AccessMode.java  | 15 +++++++
 .../main/java/org/neo4j/driver/v1/Config.java |  6 ++-
 .../main/java/org/neo4j/driver/v1/Logger.java | 39 +++++++++++++++++++
 3 files changed, 59 insertions(+), 1 deletion(-)

diff --git a/driver/src/main/java/org/neo4j/driver/v1/AccessMode.java b/driver/src/main/java/org/neo4j/driver/v1/AccessMode.java
index a3277efb97..0d343f1ed1 100644
--- a/driver/src/main/java/org/neo4j/driver/v1/AccessMode.java
+++ b/driver/src/main/java/org/neo4j/driver/v1/AccessMode.java
@@ -18,8 +18,23 @@
  */
 package org.neo4j.driver.v1;
 
+/**
+ * Used by Routing Driver to decide if a transaction should be routed to a write server or a read server in a cluster.
+ * When running a transaction, a write transaction requires a server that supports writes.
+ * A read transaction, on the other hand, requires a server that supports read operations.
+ * This classification is key for routing driver to route transactions to a cluster correctly.
+ *
+ * While any {@link AccessMode} will be ignored while running transactions via a driver towards a single server.
+ * As the single server serves both read and write operations at the same time.
+ */
 public enum AccessMode
 {
+    /**
+     * Use this for transactions that requires a read server in a cluster
+     */
     READ,
+    /**
+     * Use this for transactions that requires a write server in a cluster
+     */
     WRITE
 }
diff --git a/driver/src/main/java/org/neo4j/driver/v1/Config.java b/driver/src/main/java/org/neo4j/driver/v1/Config.java
index ce09650102..6f2a764b97 100644
--- a/driver/src/main/java/org/neo4j/driver/v1/Config.java
+++ b/driver/src/main/java/org/neo4j/driver/v1/Config.java
@@ -303,7 +303,8 @@ public ConfigBuilder withSessionLivenessCheckTimeout( long timeout )
          * application seeing connection problems, and performance.
          * <p>
          * You normally should not need to tune this parameter.
-         * This feature is turned off by default. Value {@code 0} means connections will always be tested for
+         * No connection liveliness check is done by default.
+         * Value {@code 0} means connections will always be tested for
          * validity and negative values mean connections will never be tested.
          *
          * @param value the minimum idle time in milliseconds
@@ -516,6 +517,9 @@ public enum EncryptionLevel
      */
     public static class TrustStrategy
     {
+        /**
+         * The trust strategy that the driver supports
+         */
         public enum Strategy
         {
             @Deprecated
diff --git a/driver/src/main/java/org/neo4j/driver/v1/Logger.java b/driver/src/main/java/org/neo4j/driver/v1/Logger.java
index b913320de0..ac52d2668b 100644
--- a/driver/src/main/java/org/neo4j/driver/v1/Logger.java
+++ b/driver/src/main/java/org/neo4j/driver/v1/Logger.java
@@ -23,17 +23,56 @@
  */
 public interface Logger
 {
+    /**
+     * Logs errors from this driver
+     * @param message the error message
+     * @param cause the cause of the error
+     */
     void error( String message, Throwable cause );
 
+    /**
+     * Logs information from the driver
+     * @param message the information message
+     * @param params parameters used in the information message
+     */
     void info( String message, Object... params );
 
+    /**
+     * Logs warnings that happened during using the driver
+     * @param message the warning message
+     * @param params parameters used in the warning message
+     */
     void warn( String message, Object... params );
 
+    /**
+     * Logs bolt messages sent and received by this driver.
+     * It is only enabled when {@link Logger#isDebugEnabled()} returns {@code True}.
+     * This logging level generates a lot of log entries.
+     * @param message the bolt message
+     * @param params parameters used in generating the bolt message
+     */
     void debug( String message, Object... params );
 
+    /**
+     * Logs binary sent and received by this driver.
+     * It is only enabled when {@link Logger#isTraceEnabled()} returns {@code True}.
+     * This logging level generates huge amount of log entries.
+     * @param message the bolt message in hex
+     * @param params parameters used in generating the hex message
+     */
     void trace( String message, Object... params );
 
+    /**
+     * Return true if the trace logging level is enabled.
+     * @see Logger#trace(String, Object...)
+     * @return true if the trace logging level is enabled.
+     */
     boolean isTraceEnabled();
 
+    /**
+     * Return true if the debug level is enabled.
+     * @see Logger#debug(String, Object...)
+     * @return true if the debug level is enabled.
+     */
     boolean isDebugEnabled();
 }