Don't synchronize on stats update in ThriftHiveMetastore

unix280 · Nov 16, 2024 · f72d863 · f72d863
1 parent ecfda83
commit f72d863
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...etastore/src/main/java/com/facebook/presto/hive/metastore/thrift/ThriftHiveMetastore.java b/...etastore/src/main/java/com/facebook/presto/hive/metastore/thrift/ThriftHiveMetastore.java
@@ -581,7 +581,7 @@ private Map<String, HiveColumnStatistics> groupStatisticsByColumn(List<ColumnSta
     }
 
     @Override
-    public synchronized void updateTableStatistics(MetastoreContext metastoreContext, String databaseName, String tableName, Function<PartitionStatistics, PartitionStatistics> update)
+    public void updateTableStatistics(MetastoreContext metastoreContext, String databaseName, String tableName, Function<PartitionStatistics, PartitionStatistics> update)
     {
         PartitionStatistics currentStatistics = getTableStatistics(metastoreContext, databaseName, tableName);
         PartitionStatistics updatedStatistics = update.apply(currentStatistics);
@@ -652,7 +652,7 @@ private void deleteTableColumnStatistics(MetastoreContext metastoreContext, Stri
     }
 
     @Override
-    public synchronized void updatePartitionStatistics(MetastoreContext metastoreContext, String databaseName, String tableName, String partitionName, Function<PartitionStatistics, PartitionStatistics> update)
+    public void updatePartitionStatistics(MetastoreContext metastoreContext, String databaseName, String tableName, String partitionName, Function<PartitionStatistics, PartitionStatistics> update)
     {
         PartitionStatistics currentStatistics = requireNonNull(
                 getPartitionStatistics(metastoreContext, databaseName, tableName, ImmutableSet.of(partitionName)).get(partitionName), "getPartitionStatistics() returned null");