(polaris) branch main updated: Do not log errors for propagated exceptions in IcebergCatalogAdapter (#876)

[dimas]

This is an automated email from the ASF dual-hosted git repository.

dimas pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new 7d854f2f Do not log errors for propagated exceptions in 
IcebergCatalogAdapter (#876)
7d854f2f is described below

commit 7d854f2fc50b67b325b624bf7681744385dc82ab
Author: Dmitri Bourlatchkov <dmitri.bourlatch...@dremio.com>
AuthorDate: Fri Jan 24 17:24:49 2025 -0500

    Do not log errors for propagated exceptions in IcebergCatalogAdapter (#876)
    
    Log them as "DEBUG" instead.
    
    This is to avoid having an "ERROR" log message about "namespace not
    found" (with a long stack trace) when, for example, Spark executes
    a `CREATE NAMESPACE myNewNamespace`.
---
 .../java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/service/common/src/main/java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java
 
b/service/common/src/main/java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java
index a123019c..10727a7c 100644
--- 
a/service/common/src/main/java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java
+++ 
b/service/common/src/main/java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java
@@ -162,7 +162,7 @@ public class IcebergCatalogAdapter
     try (PolarisCatalogHandlerWrapper wrapper = 
newHandlerWrapper(securityContext, catalogName)) {
       return action.apply(wrapper);
     } catch (RuntimeException e) {
-      LOGGER.error("Error while operating on catalog", e);
+      LOGGER.debug("RuntimeException while operating on catalog. Propagating 
to caller.", e);
       throw e;
     } catch (Exception e) {
       LOGGER.error("Error while operating on catalog", e);