Review Solr Streaming code for typos (#2657)

Co-authored-by: Christine Poerschke <[email protected]>

Author: epugh

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/SolrClientCache.java

@@ -41,13 +41,13 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-/** The SolrClientCache caches SolrClients so they can be reused by different TupleStreams. */
+/** The SolrClientCache caches SolrClients, so they can be reused by different TupleStreams. */
 public class SolrClientCache implements Closeable {
 
   private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
 
   // Set the floor for timeouts to 60 seconds.
-  // Timeouts cans be increased by setting the system properties defined below.
+  // Timeouts can be increased by setting the system properties defined below.
   private static final int MIN_TIMEOUT = 60000;
   private static final int minConnTimeout =
       Math.max(
@@ -148,7 +148,7 @@ private static CloudHttp2SolrClient newCloudHttp2SolrClient(
   /**
    * Create (and cache) a SolrClient based around the provided URL
    *
-   * @param baseUrl a Solr URL. May be either a "base" URL (i.e. ending in "/solr"), or point to a
+   * @param baseUrl a Solr URL. May either be a "base" URL (i.e. ending in "/solr"), or point to a
    *     particular collection or core.
    * @return a SolrClient configured to use the provided URL. The cache retains a reference to the
    *     returned client, and will close it when callers invoke {@link SolrClientCache#close()}

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/comp/FieldComparator.java

@@ -98,13 +98,13 @@ public Explanation toExplanation(StreamFactory factory) throws IOException {
 
   /*
    * What are we doing here messing around with lambdas for the comparator logic?
-   * We want the compare(...) function to run as fast as possible because it will be called many many
+   * We want the compare(...) function to run as fast as possible because it will be called many
    * times over the lifetime of this object. For that reason we want to limit the number of comparisons
    * taking place in the compare(...) function. Because this class supports both ascending and
    * descending comparisons and the logic for each is slightly different, we want to do the
    *   if(ascending){ compare like this } else { compare like this }
    * check only once - we can do that in the constructor of this class, create a lambda, and then execute
-   * that lambda in the compare function. A little bit of branch prediction savings right here.
+   * that lambda in the compare function. A bit of branch prediction savings right here.
    */
   @SuppressWarnings({"unchecked"})
   private void assignComparator() {

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/BicubicSplineEvaluator.java

@@ -36,7 +36,7 @@ public Object doWork(Object... objects) throws IOException {
 
     if (objects.length != 3) {
       throw new IOException(
-          "The bicubicSpline function requires three paremeters," + objects.length + " found.");
+          "The bicubicSpline function requires three parameters," + objects.length + " found.");
     }
 
     Object first = objects[0];

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/IsNullEvaluator.java

@@ -45,7 +45,7 @@ public Object doWork(Object... values) throws IOException {
     }
 
     if (values[0] instanceof String) {
-      // Check to see if the this tuple had a null value for that string.
+      // Check to see if this tuple had a null value for that string.
       String nullField = getStreamContext().getTupleContext().get("null");
       if (nullField != null && nullField.equals(values[0])) {
         return true;

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/KnnRegressionEvaluator.java

@@ -58,7 +58,7 @@ public Object doWork(Object... values) throws IOException {
 
     if (values.length < 3) {
       throw new IOException(
-          "knnRegress expects atleast three parameters: an observation matrix, an outcomes vector and k.");
+          "knnRegress expects at least three parameters: an observation matrix, an outcomes vector and k.");
     }
 
     Matrix observations = null;

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/Matrix.java

@@ -61,8 +61,8 @@ public List<String> getRowLabels() {
     return rowLabels;
   }
 
-  public void setRowLabels(List<String> rowLables) {
-    this.rowLabels = rowLables;
+  public void setRowLabels(List<String> rowLabels) {
+    this.rowLabels = rowLabels;
   }
 
   public double[][] getData() {

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/NotNullEvaluator.java

@@ -45,7 +45,7 @@ public Object doWork(Object... values) throws IOException {
     }
 
     if (values[0] instanceof String) {
-      // Check to see if the this tuple had a null value for that string.
+      // Check to see if this tuple had a null value for that string.
       String nullField = getStreamContext().getTupleContext().get("null");
       if (nullField != null && nullField.equals(values[0])) {
         return false;

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/PredictEvaluator.java

@@ -178,7 +178,7 @@ public Object doWork(Object... objects) throws IOException {
           y = ((Number) third).doubleValue();
           return bivariateFunction.value(x, y);
         } else {
-          throw new IOException("BivariateFunction requires two numberic parameters.");
+          throw new IOException("BivariateFunction requires two numeric parameters.");
         }
       } else if (objects.length == 2) {
         if (second instanceof Matrix) {

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/ProbabilityEvaluator.java

@@ -60,7 +60,7 @@ public Object doWork(Object... values) throws IOException {
         throw new IOException(
             String.format(
                 Locale.ROOT,
-                "Invalid expression %s - found type %s for the first value, expecting a IntegerDistributionm for probability at a specific value.",
+                "Invalid expression %s - found type %s for the first value, expecting a IntegerDistribution for probability at a specific value.",
                 toExpression(constructingFactory),
                 first.getClass().getSimpleName()));
       }

solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/eval/RecursiveEvaluator.java

@@ -126,7 +126,7 @@ protected Object normalizeOutputType(Object value) {
       return ((List<?>) value)
           .stream().map(innerValue -> normalizeOutputType(innerValue)).collect(Collectors.toList());
     } else if (value instanceof Tuple && value.getClass().getEnclosingClass() == null) {
-      // If its a tuple and not a inner class that has extended tuple, which is done in a number of
+      // If it's a tuple and not an inner class that has extended tuple, which occurs in a number of
       // cases so that mathematical models can be contained within a tuple.
 
       Tuple tuple = (Tuple) value;