Fix typos and spelling issues identified by codespell (#2959)

* Fix findings with codespell

* Fix findings with codespell

Author: MonkeyCanCode

CONTRIBUTING.md

@@ -123,7 +123,7 @@ Tips:
 
 The Apache Polaris build currently requires Java 21 or later. There are a few tools that help you running the right Java version:
 
-* [SDKMAN!](https://sdkman.io/) follow the installation instructions, then run `sdk list java` to see the available distributions and versions, then run `sdk install java <identifer from list>` using the identifier for the distribution and version (>= 21) of your choice.
+* [SDKMAN!](https://sdkman.io/) follow the installation instructions, then run `sdk list java` to see the available distributions and versions, then run `sdk install java <identifier from list>` using the identifier for the distribution and version (>= 21) of your choice.
 * [jenv](https://www.jenv.be/) If on a Mac you can use jenv to set the appropriate SDK.
 
 ## Code Contribution Guidelines

getting-started/spark/notebooks/SparkPolaris.ipynb

@@ -638,7 +638,7 @@
    "metadata": {},
    "source": [
     "# Add another Principal Role to the Engineer Principal\n",
-    "A principal can belong to multiple Principal Roles. Typically, a call will use the union of all privilages assigned to all of the principal's roles. "
+    "A principal can belong to multiple Principal Roles. Typically, a call will use the union of all privileges assigned to all of the principal's roles. "
    ]
   },
   {

helm/polaris/templates/_helpers.tpl

@@ -157,7 +157,7 @@ line breaks, they will be escaped and a multi-line option will be printed.
 Escapes a property key to be used in a configmap, conforming with the Java parsisng rules for
 property files: https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Properties.html#load(java.io.Reader)
 - Escapes all backslashes.
-- Escapes all key termination charaters: '=', ':' and whitespace.
+- Escapes all key termination characters: '=', ':' and whitespace.
 */}}
 {{- define "polaris.escapeConfigOptionKey" -}}
 {{- $key := . -}}
@@ -204,7 +204,7 @@ Converts a Kubernetes quantity to a number (int64 if possible or float64 otherwi
 It handles raw numbers as well as quantities with suffixes
 like m, k, M, G, T, P, E, ki, Mi, Gi, Ti, Pi, Ei.
 It also handles scientific notation.
-Quantities should be positive, so negative values, zero, or any unparseable number
+Quantities should be positive, so negative values, zero, or any unparsable number
 will result in a failure.
 https://kubernetes.io/docs/reference/kubernetes-api/common-definitions/quantity/
 */}}

integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationBase.java

@@ -1028,7 +1028,7 @@ public void multipleDiffsAgainstMultipleTablesLastFails() {
     Transaction t2Transaction = catalog().loadTable(identifier2).newTransaction();
     t2Transaction.updateSchema().renameColumn("data", "new-column").commit();
 
-    // delete the colum that is being renamed in the above TX to cause a conflict
+    // delete the column that is being renamed in the above TX to cause a conflict
     table2.updateSchema().deleteColumn("data").commit();
     Schema updatedSchemaTwo = table2.schema();
 

persistence/nosql/idgen/README.md

@@ -42,7 +42,7 @@ Node IDs are leased by every "JVM running Polaris" for a period of time. The ID
 that no IDs will be generated for a timestamp that exceeds the "lease time". Leases can be extended. The implementation
 leverages atomic database operations (CAS) for the lease implementation.
 
-ID generators must not use timestamps before or after the lease period nor must they re-use an older timestamp. This
+ID generators must not use timestamps before or after the lease period nor must they reuse an older timestamp. This
 requirement is satisfied using a monotonic clock implementation.
 
 ## Code structure

plugins/spark/v3.5/getting-started/notebooks/SparkPolaris.ipynb

@@ -399,7 +399,7 @@
     "    id int, name string)\n",
     "USING delta LOCATION 'file:///tmp/delta_tables/people';\n",
     "\"\"\")\n",
-    "# You can also use cloud storage like s3. For eample: s3://<bucket-name>/<path-to-table>\n",
+    "# You can also use cloud storage like s3. For example: s3://<bucket-name>/<path-to-table>\n",
     "# Make the corresponding credentials are set up correctly"
    ]
   },

plugins/spark/v3.5/integration/src/intTest/java/org/apache/polaris/spark/quarkus/it/SparkDeltaIT.java

@@ -236,7 +236,7 @@ public void testDataframeSaveOperations() {
     List<String> subDirs = listDirs(getTableLocation(deltatb));
     assertThat(subDirs).contains("_delta_log");
 
-    // verify we can create a table out of the exising delta location
+    // verify we can create a table out of the existing delta location
     sql("CREATE TABLE %s USING DELTA LOCATION '%s'", deltatb, getTableLocation(deltatb));
     List<Object[]> tables = sql("SHOW TABLES");
     assertThat(tables.size()).isEqualTo(1);

plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/utils/PolarisCatalogUtils.java

@@ -51,8 +51,8 @@ public static boolean useDelta(String provider) {
   }
 
   /**
-   * For tables whose location is manged by Spark Session Catalog, there will be no location or path
-   * in the properties.
+   * For tables whose location is managed by Spark Session Catalog, there will be no location or
+   * path in the properties.
    */
   public static boolean isTableWithSparkManagedLocation(Map<String, String> properties) {
     boolean hasLocationClause = properties.containsKey(TableCatalog.PROP_LOCATION);

polaris-core/src/main/java/org/apache/polaris/core/auth/PolarisAuthorizerImpl.java

@@ -865,7 +865,7 @@ public boolean isAuthorized(
    * permissions matching {@code privilege} on any entity in the resolvedPath of the resolvedPath.
    *
    * <p>The caller is responsible for translating these checks into either behavioral actions (e.g.
-   * returning 404 instead of 403, checking other root privileges that supercede the checked
+   * returning 404 instead of 403, checking other root privileges that supersede the checked
    * privilege, choosing whether to vend credentials) or throwing relevant Unauthorized
    * errors/exceptions.
    */

polaris-core/src/main/java/org/apache/polaris/core/config/BehaviorChangeConfiguration.java

@@ -70,7 +70,7 @@ protected BehaviorChangeConfiguration(
               .key("TABLE_OPERATIONS_MAKE_METADATA_CURRENT_ON_COMMIT")
               .description(
                   "If true, BasePolarisTableOperations should mark the metadata that is passed into"
-                      + " `commit` as current, and re-use it to skip a trip to object storage to re-construct"
+                      + " `commit` as current, and reuse it to skip a trip to object storage to re-construct"
                       + " the committed metadata again.")
               .defaultValue(true)
               .buildBehaviorChangeConfiguration();