Skip serialization on explain query

Author: albertyw

project/tests/test_execute_sql.py

@@ -131,7 +131,6 @@ def test_explain_simple(self):
         mock_cursor = sql.connection.cursor.return_value.__enter__.return_value
         sql.connection.ops.explain_query_prefix.return_value = prefix
         execute_sql(sql)
-        params = tuple(force_str(param) for param in params)
         mock_cursor.execute.assert_called_once_with(f"{prefix} {_simple_mock_query_sql}", params)
 
     def test_explain_unicode(self):
@@ -141,7 +140,6 @@ def test_explain_unicode(self):
         mock_cursor = sql.connection.cursor.return_value.__enter__.return_value
         sql.connection.ops.explain_query_prefix.return_value = prefix
         execute_sql(sql)
-        params = tuple(force_str(param) for param in params)
         mock_cursor.execute.assert_called_once_with(f"{prefix} {_simple_mock_query_sql}", params)
 
     def test_explain_non_unicode(self):
@@ -151,4 +149,4 @@ def test_explain_non_unicode(self):
         mock_cursor = sql.connection.cursor.return_value.__enter__.return_value
         sql.connection.ops.explain_query_prefix.return_value = prefix
         execute_sql(sql)
-        self.assertFalse(mock_cursor.execute.called)
+        mock_cursor.execute.assert_called_once_with(f"{prefix} {_simple_mock_query_sql}", params)

silk/sql.py

@@ -55,18 +55,7 @@ def _explain_query(connection, q, params):
         # for queries other than `select`
         prefixed_query = f"{prefix} {q}"
         with connection.cursor() as cur:
-            try:
-                params_str = tuple(force_str(param) for param in params)
-            except UnicodeDecodeError:
-                # Sometimes `force_str` can still raise a UnicodeDecodeError
-                # Reference: https://github.com/jazzband/django-silk/issues?q=encoding
-                Logger.error(
-                    "UnicodeDecodeError while trying to explain query: %s. "
-                    "This could be caused by a non-UTF-8 encoded parameter.",
-                    q
-                )
-                return None
-            cur.execute(prefixed_query, params_str)
+            cur.execute(prefixed_query, params)
             result = _unpack_explanation(cur.fetchall())
             return '\n'.join(result)
     return None