fix: Do not update structure version on build range end update (#6015)

Author: paveltiunov

packages/cubejs-query-orchestrator/src/orchestrator/PreAggregations.ts

@@ -90,7 +90,7 @@ export function getLastUpdatedAtTimestamp(
 }
 
 function getStructureVersion(preAggregation) {
-  const versionArray = [preAggregation.loadSql];
+  const versionArray = [preAggregation.structureVersionLoadSql || preAggregation.loadSql];
   if (preAggregation.indexesSql && preAggregation.indexesSql.length) {
     versionArray.push(preAggregation.indexesSql);
   }
@@ -179,6 +179,7 @@ export type PreAggregationDescription = {
   timezone: string;
   indexesSql: IndexDescription[];
   invalidateKeyQueries: QueryWithParams[];
+  structureVersionLoadSql: QueryWithParams;
   sql: QueryWithParams;
   loadSql: QueryWithParams;
   tableName: string;
@@ -1575,6 +1576,8 @@ export class PreAggregationPartitionRangeLoader {
     return {
       ...this.preAggregation,
       tableName: partitionTableName,
+      structureVersionLoadSql: this.preAggregation.loadSql &&
+        this.replacePartitionSqlAndParams(this.preAggregation.loadSql, range, partitionTableName),
       loadSql: this.preAggregation.loadSql &&
         this.replacePartitionSqlAndParams(this.preAggregation.loadSql, loadRange, partitionTableName),
       sql: this.preAggregation.sql &&

packages/cubejs-query-orchestrator/test/unit/QueryOrchestrator.test.js

@@ -1214,6 +1214,7 @@ describe('QueryOrchestrator', () => {
     expect(result.data[0]).toMatch(/orders_d20210501/);
     expect(result.data[0]).not.toMatch(/orders_h2021053000/);
     expect(result.data[0]).toMatch(/orders_h2021053100/);
+    expect(result.data[0]).toMatch(/orders_h2021060100_2i3ibaj2_d004iq51/);
 
     result = await queryOrchestrator.fetchQuery(query(['2021-05-31T00:00:00.000', '2021-05-31T23:59:59.999']));
     console.log(JSON.stringify(result, null, 2));