Apply suggestions from code review

Co-authored-by: Valentin Kovalenko <[email protected]>

Author: katcharov

driver-core/src/main/com/mongodb/connection/AsyncTransportSettings.java

@@ -22,19 +22,19 @@
 import java.util.concurrent.ExecutorService;
 
 /**
- * {@link TransportSettings} for an async transport implementation.
+ * {@link TransportSettings} for a non-<a href="http://netty.io/">Netty</a>-based async transport implementation.
  *
  * @since 5.2
  */
-public class AsyncTransportSettings extends TransportSettings {
+public final class AsyncTransportSettings extends TransportSettings {
 
     private final ExecutorService executorService;
 
-    public AsyncTransportSettings(final Builder builder) {
+    private AsyncTransportSettings(final Builder builder) {
         this.executorService = builder.executorService;
     }
 
-    public static Builder builder() {
+    static Builder builder() {
         return new Builder();
     }
 
@@ -54,6 +54,7 @@ private Builder() {
          * @param executorService the executor service
          * @return this
          * @see #getExecutorService()
+         * @see AsynchronousChannelGroup#withThreadPool(ExecutorService)
          */
         public Builder executorService(final ExecutorService executorService) {
             this.executorService = executorService;

driver-core/src/main/com/mongodb/internal/connection/AsynchronousSocketChannelStreamFactory.java

@@ -49,7 +49,7 @@ public AsynchronousSocketChannelStreamFactory(
         this(inetAddressResolver, settings, sslSettings, null);
     }
 
-    public AsynchronousSocketChannelStreamFactory(
+    AsynchronousSocketChannelStreamFactory(
             final InetAddressResolver inetAddressResolver, final SocketSettings settings,
             final SslSettings sslSettings, @Nullable final ExecutorService executorService) {
         assertFalse(sslSettings.isEnabled());

driver-core/src/main/com/mongodb/internal/connection/AsynchronousSocketChannelStreamFactoryFactory.java

@@ -37,7 +37,7 @@ public AsynchronousSocketChannelStreamFactoryFactory(final InetAddressResolver i
         this(inetAddressResolver, null);
     }
 
-    public AsynchronousSocketChannelStreamFactoryFactory(
+    AsynchronousSocketChannelStreamFactoryFactory(
             final InetAddressResolver inetAddressResolver,
             @Nullable final ExecutorService executorService) {
         this.inetAddressResolver = inetAddressResolver;

driver-core/src/main/com/mongodb/internal/connection/TlsChannelStreamFactoryFactory.java

@@ -72,7 +72,7 @@ public class TlsChannelStreamFactoryFactory implements StreamFactoryFactory {
     /**
      * Construct a new instance
      */
-    public TlsChannelStreamFactoryFactory(final InetAddressResolver inetAddressResolver,
+    TlsChannelStreamFactoryFactory(final InetAddressResolver inetAddressResolver,
             @Nullable final ExecutorService executorService) {
         this.inetAddressResolver = inetAddressResolver;
         this.group = new AsynchronousTlsChannelGroup(executorService);