fix: remove redundant hacks: primaryKey filter for method dimensionColumns (#161)

Author: itrcz

packages/cubejs-oracle-driver/driver/OracleDriver.js

@@ -19,13 +19,6 @@ const reduceCb = (result, i) => {
 
   if (i.key_type === "P" || i.key_type === "U") {
     attributes.push(["primaryKey"]);
-    /**
-     * ScaffoldingSchema.js line 139
-     * seems like a bug
-     * TODO: remove this after method
-     * dimensionColumns change
-     */
-    attributes.primaryKey = true;
   }
 
   tables.push({

packages/cubejs-schema-compiler/scaffolding/ScaffoldingSchema.js

@@ -136,7 +136,7 @@ class ScaffoldingSchema {
     const dimensionColumns = tableDefinition.filter(
       column =>
         !column.name.startsWith('_') && this.columnType(column) === 'string' ||
-        column.attributes && column.attributes.primaryKey ||
+        column.attributes && column.attributes.indexOf('primaryKey') !== -1 ||
         column.name.toLowerCase() === 'id'
     );