feat(cubestore-driver): Support timestamp(3) as a column type (#6324)

Author: ovr

.github/workflows/cloud.yml

@@ -1,21 +1,25 @@
 # CAUTION: This workflow exposes secrets. It is only supposed to be run on "merge into master" condition.
 name: 'Cloud Tests'
 on:
+  # push doesn't work for PRs, it's safe to use it
+  # only maintainers/core contributors are allowed to push directly to the main repository
   push:
     paths:
-      - '.github/actions/integration/**'
+      - '.github/actions/integration/athena.sh'
+      - '.github/actions/integration/bigquery.sh'
       - '.github/workflows/cloud.yml'
-      - 'packages/**'
-      - '.eslintrc.js'
-      - '.prettierrc'
+      - 'packages/cubejs-testing/**'
+      - 'packages/cubejs-testing-shared/**'
+      - 'packages/cubejs-backend-shared/**'
+      - 'packages/cubejs-base-driver/**'
+      - 'packages/cubejs-query-orchestrator/**'
+      - 'packages/cubejs-schema-compiler/**'
+      # drivers
+      - 'packages/cubejs-athena-driver/**'
+      - 'packages/cubejs-bigquery-driver/**'
       - 'lerna.json'
       - 'package.json'
-      - 'rollup.config.js'
       - 'yarn.lock'
-      - 'docs-gen/**'
-      - 'docs/**'
-    branches:
-      - master
 
 jobs:
   integration-cloud:

packages/cubejs-athena-driver/src/AthenaDriver.ts

@@ -20,7 +20,7 @@ import {
   DownloadTableCSVData,
   DriverInterface,
   QueryOptions, StreamOptions,
-  StreamTableData, TableName
+  StreamTableData
 } from '@cubejs-backend/base-driver';
 import * as SqlString from 'sqlstring';
 import { AthenaClientConfig } from '@aws-sdk/client-athena/dist-types/AthenaClient';

packages/cubejs-cubestore-driver/src/CubeStoreDriver.ts

@@ -24,7 +24,10 @@ import { WebSocketConnection } from './WebSocketConnection';
 const GenericTypeToCubeStore: Record<string, string> = {
   string: 'varchar(255)',
   text: 'varchar(255)',
-  uuid: 'varchar(64)'
+  uuid: 'varchar(64)',
+  // Cube Store uses an old version of sql parser which doesn't support timestamp with custom precision, but
+  // athena driver (I believe old version) allowed to use it
+  'timestamp(3)': 'timestamp',
 };
 
 type Column = {