addressing reviews

geode-assembly/src/acceptanceTest/java/org/apache/geode/client/sni/ClientSNIAcceptanceTest.java

@@ -91,12 +91,17 @@ public void connectToSNIProxyDocker() {
         .setPoolSocketFactory(ProxySocketFactories.sni("localhost",
             proxyPort))
         .create();
+    // the geode-starter.gfsh script has created a Region named "jellyfish" on the
+    // server sitting behind the haproxy gateway. Show that an empty client cache can
+    // put something in that region and then retrieve it.
     Region<String, String> region =
         cache.<String, String>createClientRegionFactory(ClientRegionShortcut.PROXY)
             .create("jellyfish");
     region.destroy("hello");
     region.put("hello", "world");
     assertThat(region.get("hello")).isEqualTo("world");
+    // the geode-starter.gfsh script put an entry named "foo" into the region with the
+    // value "bar"
     assertThat(region.get("foo")).isEqualTo("bar");
   }
 }

geode-docs/reference/topics/client-cache.html.md.erb

@@ -340,7 +340,7 @@ Provide a server list or `locator` list, but not both.
 ```
 ## <a id="cc-socket-factory" class="no-quick-link"></a>&lt;socket-factory&gt;
 
-Defines a factory to create socket connections to locators and servers.  A typical use of this element is to redirect connections to an ingress gateway such as Istio or HAProxy in a cluster where the TLS (SSL) Server Name Extension field is set to indicate the actual locator or server the client is trying to reach.  This allows you to expose only the gateway hostname:port without the client needing to be able to resolve the names of the locator and server machines.
+Defines a factory to create socket connections to locators and servers.  A typical use of this element is to redirect connections to an ingress gateway such as Istio or HAProxy in a cluster where the TLS (SSL) Server Name Indication (SNI)field is set to indicate the actual locator or server the client is trying to reach.  This allows you to expose only the gateway hostname:port without the client needing to be able to resolve the names of the locator and server machines.
 
 **Note:**
 This setting may be used with either a Server list or a Locator list.  It will be used to form connections to either.