Don't pass expected_parameter_key/value with Hive Locking enabled

Author: laithalzyoud

plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/catalog/hms/HiveMetastoreTableOperations.java

@@ -113,10 +113,13 @@ private void commitTableUpdate(Table table, TableMetadata metadata, BiFunction<T
 
             Table updatedTable = tableUpdateFunction.apply(table, newMetadataLocation);
 
+            // Passing environment context causes redundant operations if Hive locking is enabled
+            Map<String, String> environmentContext = lockingEnabled ? ImmutableMap.of() : environmentContext(metadataLocation);
+
             // todo privileges should not be replaced for an alter
             PrincipalPrivileges privileges = table.getOwner().map(MetastoreUtil::buildInitialPrivilegeSet).orElse(NO_PRIVILEGES);
             try {
-                metastore.replaceTable(table.getDatabaseName(), table.getTableName(), updatedTable, privileges, environmentContext(metadataLocation));
+                metastore.replaceTable(table.getDatabaseName(), table.getTableName(), updatedTable, privileges, environmentContext);
             }
             catch (RuntimeException e) {
                 // Cannot determine whether the `replaceTable` operation was successful,