Fix getFieldType by using hash.id instead

Author: casab

packages/cubejs-schema-compiler/src/adapter/ClickHouseQuery.ts

@@ -155,8 +155,12 @@ export class ClickHouseQuery extends BaseQuery {
     return null;
   }
 
-  public getFieldType(id) {
-    const field = this.getField(id);
+  public getFieldType(hash) {
+    if (!hash || !hash.id) {
+      return null;
+    }
+
+    const field = this.getField(hash.id);
 
     if (field) {
       return field.definition().type;

packages/cubejs-schema-compiler/test/integration/clickhouse/clickhouse-dataschema-compiler.test.ts

@@ -411,15 +411,12 @@ describe('ClickHouse DataSchemaCompiler', () => {
     });
     logSqlAndParams(query);
 
-    const collation = query.getCollation();
-    expect(collation).toEqual('en');
-
     const sqlAndParams = query.buildSqlAndParams();
     const res = await dbRunner.testQuery(sqlAndParams);
     const sql = sqlAndParams[0];
     expect(sql).toMatch('ORDER BY `visitors__source` ASC COLLATE \'en\'');
 
-    expect(res).toEqual([{ visitors__source: 'google' }, { visitors__source: 'Gork' }, { visitors__source: 'some' }]);
+    expect(res).toEqual([{ visitors__source: 'google' }, { visitors__source: 'Gork' }, { visitors__source: 'some' }, { visitors__source: null }]);
   });
 
   it('export import', () => {