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

maximebeauchemin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new bc3e19d0a2 fix(import): Ensure import exceptions are logged (#32410)
bc3e19d0a2 is described below

commit bc3e19d0a2bbf1d83d6d052507ede757eea960d3
Author: Paul Rhodes <[email protected]>
AuthorDate: Tue Mar 18 19:35:57 2025 +0000

    fix(import): Ensure import exceptions are logged (#32410)
---
 superset/commands/importers/v1/__init__.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/superset/commands/importers/v1/__init__.py 
b/superset/commands/importers/v1/__init__.py
index 989c494b60..be3bb92d0e 100644
--- a/superset/commands/importers/v1/__init__.py
+++ b/superset/commands/importers/v1/__init__.py
@@ -14,6 +14,7 @@
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
+import logging
 from typing import Any, Optional
 
 from marshmallow import Schema, validate  # noqa: F401
@@ -34,6 +35,8 @@ from superset.daos.base import BaseDAO
 from superset.models.core import Database  # noqa: F401
 from superset.utils.decorators import transaction
 
+logger = logging.getLogger(__name__)
+
 
 class ImportModelsCommand(BaseCommand):
     """Import models"""
@@ -104,6 +107,8 @@ class ImportModelsCommand(BaseCommand):
         self._prevent_overwrite_existing_model(exceptions)
 
         if exceptions:
+            for ex in exceptions:
+                logger.warning("Import Error: %s", ex)
             raise CommandInvalidError(
                 f"Error importing {self.model_name}",
                 exceptions,

Reply via email to