From 1c76a49ef8062b2069d565f3a99fccd41624fe74 Mon Sep 17 00:00:00 2001
From: "Maxim.Kolmakov" <maxim.kolmakov@jetbrains.com>
Date: Mon, 16 Dec 2024 17:16:34 +0100
Subject: [PATCH] Add filter to selectors for Compare Modes

Limit the fetching to 1 month
---
 .../common/compare/CompareModes.vue           | 34 +++++++++++++------
 pkg/server/clickhouse.go                      |  2 +-
 2 files changed, 24 insertions(+), 12 deletions(-)

diff --git a/dashboard/new-dashboard/src/components/common/compare/CompareModes.vue b/dashboard/new-dashboard/src/components/common/compare/CompareModes.vue
index 840443f7..3f71523d 100644
--- a/dashboard/new-dashboard/src/components/common/compare/CompareModes.vue
+++ b/dashboard/new-dashboard/src/components/common/compare/CompareModes.vue
@@ -2,13 +2,11 @@
   <div class="flex flex-col gap-5">
     <StickyToolbar>
       <template #start>
-        <div class="flex items-center">
-          <MachineSelect :machine-configurator="machineConfigurator" />
-        </div>
         <BranchSelect
           :branch-configurator="branchConfigurator"
           :triggered-by-configurator="triggeredByConfigurator"
         />
+        <MachineSelect :machine-configurator="machineConfigurator" />
         <DimensionSelect
           label="Mode"
           :dimension="testModeConfigurator1"
@@ -124,6 +122,7 @@ import DimensionSelect from "../../charts/DimensionSelect.vue"
 import { modeSelectLabelFormat } from "../../../shared/labels"
 import { createTestModeConfigurator } from "../../../configurators/TestModeConfigurator"
 import BranchSelect from "../BranchSelect.vue"
+import { TimeRangeConfigurator } from "../../../configurators/TimeRangeConfigurator"
 
 interface CompareBranchesProps {
   dbName: string
@@ -158,22 +157,34 @@ const persistentStateManager = new PersistentStateManager(
   },
   router
 )
-const machineConfigurator = new MachineConfigurator(serverConfigurator, persistentStateManager)
-const branchConfigurator = createBranchConfigurator(serverConfigurator, persistentStateManager, [])
-const triggeredByConfigurator = privateBuildConfigurator(serverConfigurator, persistentStateManager, [])
+
+const timeRangeConfigurator = new TimeRangeConfigurator(persistentStateManager)
+const branchConfigurator = createBranchConfigurator(serverConfigurator, persistentStateManager, [timeRangeConfigurator])
+const triggeredByConfigurator = privateBuildConfigurator(serverConfigurator, persistentStateManager, [timeRangeConfigurator, branchConfigurator])
+const machineConfigurator = new MachineConfigurator(serverConfigurator, persistentStateManager, [timeRangeConfigurator, branchConfigurator])
 
 const measureConfigurator = new SimpleMeasureConfigurator("metrics", persistentStateManager)
 measureConfigurator.initData(metricsNames)
 const testConfigurator = new SimpleMeasureConfigurator("tests", persistentStateManager)
 
-
-const testModeConfigurator1 = createTestModeConfigurator(serverConfigurator, persistentStateManager, [branchConfigurator], "mode1", false)
-const testModeConfigurator2 = createTestModeConfigurator(serverConfigurator, persistentStateManager, [branchConfigurator], "mode2", false)
+const testModeConfigurator1 = createTestModeConfigurator(
+  serverConfigurator,
+  persistentStateManager,
+  [timeRangeConfigurator, branchConfigurator, machineConfigurator, triggeredByConfigurator],
+  "mode1",
+  false
+)
+const testModeConfigurator2 = createTestModeConfigurator(
+  serverConfigurator,
+  persistentStateManager,
+  [timeRangeConfigurator, branchConfigurator, machineConfigurator, triggeredByConfigurator],
+  "mode2",
+  false
+)
 
 const mode1 = ref<string | null>(null)
 const mode2 = ref<string | null>(null)
 
-
 const tableData = ref<TableRow[]>()
 const fetchedData = ref<TableRow[]>()
 combineLatest([testModeConfigurator1.createObservable(), testModeConfigurator2.createObservable(), serverConfigurator.createObservable(), machineConfigurator.createObservable()])
@@ -251,7 +262,7 @@ function getColorForBuild(build1: number, build2: number) {
 }
 
 function getAllMetricsFromMode(machineConfigurator: MachineConfigurator, mode: string | null, metricNames: string[]): Observable<Result[]> {
-  if(mode == "default") {
+  if (mode == "default") {
     mode = ""
   }
   const params = {
@@ -287,6 +298,7 @@ function handleNavigateToTest(project: string, metric: string) {
 .p-button-icon-only.p-button-rounded {
   @apply text-black;
 }
+
 .lower {
   font-weight: 700;
   color: #ff5252;
diff --git a/pkg/server/clickhouse.go b/pkg/server/clickhouse.go
index 69416984..7add7a85 100644
--- a/pkg/server/clickhouse.go
+++ b/pkg/server/clickhouse.go
@@ -124,7 +124,7 @@ func (t *StatsServer) getModeComparison(request *http.Request) (*bytebufferpool.
 	}
 	measureNamesString := strings.Join(quotedMeasureNames, ",")
 
-	sql := fmt.Sprintf("SELECT project as Project, measure_name as MeasureName, arraySlice(groupArray(measure_value), 1, 50) AS MeasureValues FROM (SELECT project, measures.name as measure_name, measures.value as measure_value FROM %s ARRAY JOIN measures WHERE mode = '%s' AND branch = '%s' AND measure_name in (%s) AND machine like '%s' ORDER BY generated_time DESC)GROUP BY project, measure_name;", params.Table, params.Mode, params.Branch, measureNamesString, params.Machine)
+	sql := fmt.Sprintf("SELECT project as Project, measure_name as MeasureName, arraySlice(groupArray(measure_value), 1, 50) AS MeasureValues FROM (SELECT project, measures.name as measure_name, measures.value as measure_value FROM %s ARRAY JOIN measures WHERE mode = '%s' AND branch = '%s' AND measure_name in (%s) AND machine like '%s' AND generated_time >subtractMonths(now(),1) ORDER BY generated_time DESC)GROUP BY project, measure_name;", params.Table, params.Mode, params.Branch, measureNamesString, params.Machine)
 	db, err := t.openDatabaseConnection()
 	defer func(db driver.Conn) {
 		_ = db.Close()