Logging failures to refresh metadata (#385)

## Summary
problem - when an exception throws under
`super.refreshFromMetadataLocation` , there is no failure log which
shows how long it took to fail. Adding logging here for the total
duration Helps us measure how long a request takes when it fails to
retrieve metadata from hdfs. This is a common cause for 504 gateway
timeouts. Without this, we can only guess the total duration by
correlating it with metrics. The metrics don’t have a failure/success
dimension, and also cannot be isolated to a single request/table
metadata file.

solution - try/catch and log before throwing the existing error. There
is another PR which aims to reduce the total duration via better
configured timeouts+retries at hdfs layer
#386

At the same time, adding a missing log message for catalog update, for
future operations

## Changes

- [ ] Client-facing API Changes
- [ ] Internal API Changes
- [X] Bug Fixes
- [ ] New Features
- [ ] Performance Improvements
- [ ] Code Style
- [ ] Refactoring
- [ ] Documentation
- [ ] Tests


## Testing Done
existing tests should cover this 

- [ ] Manually Tested on local docker setup. Please include commands
ran, and their output.
- [ ] Added new tests for the changes made.
- [ ] Updated existing tests to reflect the changes made.
- [X] No tests added or updated. Please explain why. If unsure, please
feel free to ask for help.
- [ ] Some other form of testing like staging or soak time in
production. Please explain.

# Additional Information

- [ ] Breaking Changes
- [ ] Deprecations
- [ ] Large PR broken into smaller PRs, and PR plan linked in the
description.

Author: cbb330

iceberg/openhouse/internalcatalog/src/main/java/com/linkedin/openhouse/internal/catalog/OpenHouseInternalTableOperations.java

@@ -127,16 +127,25 @@ protected void refreshMetadata(final String metadataLoc) {
     long startTime = System.currentTimeMillis();
     boolean needToReload = !Objects.equal(currentMetadataLocation(), metadataLoc);
     Runnable r = () -> super.refreshFromMetadataLocation(metadataLoc);
-    if (needToReload) {
-      metricsReporter.executeWithStats(
-          r, InternalCatalogMetricsConstant.METADATA_RETRIEVAL_LATENCY, getCatalogMetricTags());
-    } else {
-      r.run();
+    try {
+      if (needToReload) {
+        metricsReporter.executeWithStats(
+            r, InternalCatalogMetricsConstant.METADATA_RETRIEVAL_LATENCY, getCatalogMetricTags());
+      } else {
+        r.run();
+      }
+      log.info(
+          "refreshMetadata from location {} succeeded, took {} ms",
+          metadataLoc,
+          System.currentTimeMillis() - startTime);
+    } catch (Exception e) {
+      log.error(
+          "refreshMetadata from location {} failed after {} ms",
+          metadataLoc,
+          System.currentTimeMillis() - startTime,
+          e);
+      throw e;
     }
-    log.info(
-        "refreshMetadata from location {} took {} ms",
-        metadataLoc,
-        System.currentTimeMillis() - startTime);
   }
 
   /**
@@ -284,11 +293,26 @@ protected void doCommit(TableMetadata base, TableMetadata metadata) {
       }
 
       final TableMetadata updatedMtDataRef = updatedMetadata;
-      metricsReporter.executeWithStats(
-          () ->
-              TableMetadataParser.write(updatedMtDataRef, io().newOutputFile(newMetadataLocation)),
-          InternalCatalogMetricsConstant.METADATA_UPDATE_LATENCY,
-          getCatalogMetricTags());
+      long metadataUpdateStartTime = System.currentTimeMillis();
+      try {
+        metricsReporter.executeWithStats(
+            () ->
+                TableMetadataParser.write(
+                    updatedMtDataRef, io().newOutputFile(newMetadataLocation)),
+            InternalCatalogMetricsConstant.METADATA_UPDATE_LATENCY,
+            getCatalogMetricTags());
+        log.info(
+            "updateMetadata to location {} succeeded, took {} ms",
+            newMetadataLocation,
+            System.currentTimeMillis() - metadataUpdateStartTime);
+      } catch (Exception e) {
+        log.error(
+            "updateMetadata to location {} failed after {} ms",
+            newMetadataLocation,
+            System.currentTimeMillis() - metadataUpdateStartTime,
+            e);
+        throw e;
+      }
 
       houseTable = houseTableMapper.toHouseTable(updatedMetadata, fileIO);
       if (base != null