[SEA-NodeJS] map kernel SqlError to OperationStateError for Thrift parity

Server-reported SQL execution failures (kernel `SqlError` — bad query,
missing table, divide-by-zero, invalid cast, param type mismatch) were
surfaced as the base `HiveDriverError` on the SEA/kernel path, while the
Thrift backend raises `OperationStateError(Error)` when the operation
reaches ERROR_STATE. The comparator flagged every error path as a
class mismatch (Thrift `OperationStateError` vs SEA `HiveDriverError`),
and the Python kernel connector matches Thrift here — so the divergence
was in this mapping, not the kernel.

Map `SqlError` -> `OperationStateError(OperationStateErrorCode.Error)`,
preserving the kernel message. `OperationStateError extends
HiveDriverError`, so existing `instanceof HiveDriverError` catches are
unaffected. Other code mappings (InvalidArgument -> ParameterError,
auth, network, etc.) are unchanged.

Verified against the comparator warehouse: all ERROR_PATHS cases
(table-not-found, syntax error, unresolved column, divide-by-zero,
invalid cast, param mismatch) now match the Thrift backend.

Co-authored-by: Isaac
Signed-off-by: Madhavendra Rathore <madhavendra.rathore@databricks.com>

Author: msrathore-db

lib/sea/SeaErrorMapping.ts

@@ -147,6 +147,20 @@ export function mapKernelErrorToJsError(kErr: KernelErrorShape): ErrorWithSqlSta
       error = new ParameterError(message);
       break;
 
+    case 'SqlError': {
+      // A server-reported SQL execution failure (kernel `SqlError`, e.g. a
+      // bad query, missing table, divide-by-zero, invalid cast). The Thrift
+      // backend surfaces the same situation as `OperationStateError(Error)`
+      // when the operation reaches ERROR_STATE (see DBSQLOperation), so map
+      // SqlError to the same class for backend parity. OperationStateError
+      // extends HiveDriverError, so existing `instanceof HiveDriverError`
+      // catches are unaffected.
+      const stateError = new OperationStateError(OperationStateErrorCode.Error);
+      stateError.message = message;
+      error = stateError;
+      break;
+    }
+
     // All remaining kernel ErrorCode variants map to the base driver error class.
     // M0 intentionally does not introduce new error classes; M1 may add nuance.
     case 'NotFound':
@@ -156,7 +170,6 @@ export function mapKernelErrorToJsError(kErr: KernelErrorShape): ErrorWithSqlSta
     case 'Internal':
     case 'InvalidStatementHandle':
     case 'NetworkError':
-    case 'SqlError':
       error = new HiveDriverError(message);
       break;