chore(key-value): remove redundant exception logging

[villebro]

SUMMARY

Currently the key-value commands are logging exceptions despite reraising the exception. This causes unnecessary noise on logs if the caller is anticipating a failure, which can be caught by the reraised KeyValue(Get|Create|Delete|Update)FailedError if needed. In addition, the Upsert command didn't have an exception of its own, and was not catching the KeyValueCreateFailedError exception, causing an expected ValueError to go unnoticed:

superset/superset/key_value/commands/create.py

Lines 82 to 88 in de444d4

	try:
	if isinstance(self.key, UUID):
	entry.uuid = self.key
	else:
	entry.id = self.key
	except ValueError as ex:
	raise KeyValueCreateFailedError() from ex

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[michael-s-molina]

LGTM. Thanks for the improvements!