Remove two more NOCOMMITS from SQL

They are being tracked in the sql tracker issue as release blockers.

Original commit: elastic/x-pack-elasticsearch@783544dde5

sql/net-client/src/test/java/org/elasticsearch/xpack/sql/net/client/BasicSSLServer.java

@@ -66,7 +66,6 @@ public class BasicSSLServer {
         ks.load(BasicSSLServer.class.getResourceAsStream("/ssl/server.keystore"), pass);
         kmf.init(ks, pass);
 
-        // NOCOMMIT I think we could share a key rather than trust all, right?
         TrustManager[] trustAll = new TrustManager[] {
                 new X509TrustManager() {
                     @Override

sql/net-client/src/test/java/org/elasticsearch/xpack/sql/net/client/SSLTests.java

@@ -83,7 +83,6 @@ public class SSLTests extends ESTestCase {
     }
 
     public void testSslSetup() throws Exception {
-        // NOCOMMIT this test doesn't test anything, just logs. Probably should fix that.
         SSLContext context = SSLContext.getDefault();
         SSLSocketFactory factory = context.getSocketFactory();
         SSLSocket socket = (SSLSocket) factory.createSocket();