fix(redshift-driver): fixes column order (#6068) Thanks @rdwoodring!

* fix(redshift-driver): fixes column order

* fix(redshift-driver): resolves an issue with the original fix where types was not properly returned on success paths

Author: rdwoodring

packages/cubejs-redshift-driver/src/RedshiftDriver.ts

@@ -152,11 +152,14 @@ export class RedshiftDriver extends PostgresDriver<RedshiftDriverConfiguration>
     return false;
   }
 
-  public async unload(table: string, options: UnloadOptions): Promise<DownloadTableCSVData> {
+  public async unload(tableName: string, options: UnloadOptions): Promise<DownloadTableCSVData> {
     if (!this.config.exportBucket) {
       throw new Error('Unload is not configured');
     }
 
+    const types = await this.tableColumnTypes(tableName);
+    const columns = types.map(t => t.name).join(', ');
+
     const { bucketType, bucketName, region, unloadArn, keyId, secretKey } = this.config.exportBucket;
 
     const conn = await this.pool.connect();
@@ -198,7 +201,11 @@ export class RedshiftDriver extends PostgresDriver<RedshiftDriverConfiguration>
         }
       });
 
-      const baseQuery = `UNLOAD ('SELECT * FROM ${table}') TO '${bucketType}://${bucketName}/${exportPathName}/'`;
+      const baseQuery = `
+        UNLOAD ('SELECT ${columns} FROM ${tableName}')
+        TO '${bucketType}://${bucketName}/${exportPathName}/'
+      `;
+      
       // Prefer the unloadArn if it is present
       const credentialQuery = unloadArn
         ? `iam_role '${unloadArn}'`
@@ -215,6 +222,7 @@ export class RedshiftDriver extends PostgresDriver<RedshiftDriverConfiguration>
         return {
           exportBucketCsvEscapeSymbol: this.config.exportBucketCsvEscapeSymbol,
           csvFile: [],
+          types
         };
       }
 
@@ -243,6 +251,7 @@ export class RedshiftDriver extends PostgresDriver<RedshiftDriverConfiguration>
         return {
           exportBucketCsvEscapeSymbol: this.config.exportBucketCsvEscapeSymbol,
           csvFile,
+          types
         };
       }