ci: use underscore in E2E_TABLE_SUFFIX node prefix, not hyphen

E2E tests build SQL identifiers as `..._${E2E_TABLE_SUFFIX}`.
Hyphens aren't valid in unquoted SQL identifiers, so the previous
`-node${matrix.node-version}` suffix caused all e2e jobs to fail
with INVALID_IDENTIFIER (SQLSTATE 42602):

  DROP TABLE IF EXISTS dbsql_nodejs_sdk_e2e_interval_types_<sha>-node20
                                                                ^

Use `_node<version>` instead. The underscore keeps identifiers
valid across all matrix entries.

Co-authored-by: Isaac
Signed-off-by: Vikrant Puppala <vikrant.puppala@databricks.com>

Author: vikrantpuppala

.github/workflows/main.yml

@@ -211,8 +211,10 @@ jobs:
       E2E_PATH: ${{ secrets.TEST_PECO_WAREHOUSE_HTTP_PATH }}
       E2E_ACCESS_TOKEN: ${{ secrets.DATABRICKS_TOKEN }}
       # Include Node version in the suffix so parallel matrix entries
-      # don't collide on shared E2E table names.
-      E2E_TABLE_SUFFIX: ${{ github.sha }}-node${{ matrix.node-version }}
+      # don't collide on shared E2E table names. Use underscore (not
+      # hyphen) — hyphens aren't valid in SQL unquoted identifiers,
+      # and tests build identifiers as `..._${E2E_TABLE_SUFFIX}`.
+      E2E_TABLE_SUFFIX: ${{ github.sha }}_node${{ matrix.node-version }}
       E2E_CATALOG: peco
       E2E_SCHEMA: default
       E2E_VOLUME: e2etests