chore(key-value): remove redundant exception logging

superset/key_value/commands/create.py

@@ -64,7 +64,6 @@ def run(self) -> Key:
             return self.create()
         except SQLAlchemyError as ex:
             db.session.rollback()
-            logger.exception("Error running create command")
             raise KeyValueCreateFailedError() from ex
 
     def validate(self) -> None:

superset/key_value/commands/delete.py

@@ -50,7 +50,6 @@ def run(self) -> bool:
             return self.delete()
         except SQLAlchemyError as ex:
             db.session.rollback()
-            logger.exception("Error running delete command")
             raise KeyValueDeleteFailedError() from ex
 
     def validate(self) -> None:

superset/key_value/commands/delete_expired.py

@@ -46,7 +46,6 @@ def run(self) -> None:
             self.delete_expired()
         except SQLAlchemyError as ex:
             db.session.rollback()
-            logger.exception("Error running delete command")
             raise KeyValueDeleteFailedError() from ex
 
     def validate(self) -> None:

superset/key_value/commands/get.py

@@ -52,7 +52,6 @@ def run(self) -> Any:
         try:
             return self.get()
         except SQLAlchemyError as ex:
-            logger.exception("Error running get command")
             raise KeyValueGetFailedError() from ex
 
     def validate(self) -> None:

superset/key_value/commands/update.py

@@ -66,7 +66,6 @@ def run(self) -> Optional[Key]:
             return self.update()
         except SQLAlchemyError as ex:
             db.session.rollback()
-            logger.exception("Error running update command")
             raise KeyValueUpdateFailedError() from ex
 
     def validate(self) -> None:

superset/key_value/commands/upsert.py

@@ -26,7 +26,10 @@
 from superset import db
 from superset.commands.base import BaseCommand
 from superset.key_value.commands.create import CreateKeyValueCommand
-from superset.key_value.exceptions import KeyValueUpdateFailedError
+from superset.key_value.exceptions import (
+    KeyValueCreateFailedError,
+    KeyValueUpsertFailedError,
+)
 from superset.key_value.models import KeyValueEntry
 from superset.key_value.types import Key, KeyValueResource
 from superset.key_value.utils import get_filter
@@ -66,10 +69,9 @@ def __init__(
     def run(self) -> Key:
         try:
             return self.upsert()
-        except SQLAlchemyError as ex:
+        except (KeyValueCreateFailedError, SQLAlchemyError) as ex:
             db.session.rollback()
-            logger.exception("Error running update command")
-            raise KeyValueUpdateFailedError() from ex
+            raise KeyValueUpsertFailedError() from ex
 
     def validate(self) -> None:
         pass
@@ -90,6 +92,7 @@ def upsert(self) -> Key:
             db.session.merge(entry)
             db.session.commit()
             return Key(entry.id, entry.uuid)
+
         return CreateKeyValueCommand(
             resource=self.resource,
             value=self.value,

superset/key_value/exceptions.py

@@ -46,5 +46,9 @@ class KeyValueUpdateFailedError(UpdateFailedError):
     message = _("An error occurred while updating the value.")
 
 
+class KeyValueUpsertFailedError(UpdateFailedError):
+    message = _("An error occurred while upserting the value.")
+
+
 class KeyValueAccessDeniedError(ForbiddenError):
     message = _("You don't have permission to modify the value.")