fix(kernel): distinguish HTTP and filesystem permission errors

Signed-off-by: Aakash Saravanan <aakash.saravanan@databricks.com>

Author: aakash-saravanan-db

src/databricks/sql/backend/kernel/_errors.py

@@ -162,7 +162,10 @@ def reraise_kernel_error(
     message = getattr(exc, "message", str(exc))
     new: Union[Error, OSError]
     # Local missing files have no HTTP status; remote missing files carry 404.
-    if error_code == "FILE_TRANSFER_PERMISSION_ERROR":
+    if (
+        error_code == "FILE_TRANSFER_PERMISSION_ERROR"
+        and getattr(exc, "http_status", None) is None
+    ):
         new = PermissionError(message)
     elif (
         file_transfer

tests/unit/test_kernel_client.py

@@ -2684,6 +2684,7 @@ def test_only_reserved_headers_omits_kwarg(self, monkeypatch):
         ("NotFound", None, "FILE_TRANSFER_ERROR", FileNotFoundError),
         ("NotFound", 404, "FILE_TRANSFER_ERROR", OperationalError),
         ("PermissionDenied", None, "FILE_TRANSFER_PERMISSION_ERROR", PermissionError),
+        ("PermissionDenied", 403, "FILE_TRANSFER_PERMISSION_ERROR", OperationalError),
         ("NetworkError", None, "FILE_TRANSFER_ERROR", OperationalError),
         ("Internal", None, "FILE_TRANSFER_ERROR", OperationalError),
         ("Internal", None, None, DatabaseError),