use clickhouse as a default (#5778)

pytorch · Oct 17, 2024 · 87ad4c4 · 87ad4c4
1 parent 1969a06
commit 87ad4c4
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/torchci/components/UseClickhouseProvider.tsx b/torchci/components/UseClickhouseProvider.tsx
@@ -7,7 +7,7 @@ interface UseCHContextProps {
 }
 
 const UseCHContext = createContext<UseCHContextProps>({
-  useCH: false,
+  useCH: true,
   setUseCH: () => {},
 });
 
@@ -21,7 +21,7 @@ export function UseCHContextProvider({
 }: {
   children: React.ReactNode;
 }) {
-  const [useCH, setUseCH] = usePreference("useClickHouse", undefined, false);
+  const [useCH, setUseCH] = usePreference("useClickHouse", undefined, true);
   return (
     <UseCHContext.Provider
       value={{

diff --git a/torchci/components/metrics/panels/ScalarPanel.tsx b/torchci/components/metrics/panels/ScalarPanel.tsx
@@ -69,7 +69,7 @@ export default function ScalarPanel({
   metricName,
   // Callback to decide whether the scalar value is "bad" and should be displayed red.
   badThreshold,
-  useClickHouse = false,
+  useClickHouse = true,
 }: {
   title: string;
   queryCollection?: string;

diff --git a/torchci/components/metrics/panels/TablePanel.tsx b/torchci/components/metrics/panels/TablePanel.tsx
@@ -24,7 +24,7 @@ export default function TablePanel({
   helpLink,
   // An optional flag to show the table footer
   showFooter,
-  useClickHouse = false,
+  useClickHouse = true,
 }: {
   title: string;
   queryCollection?: string;

diff --git a/torchci/components/metrics/panels/TimeSeriesPanel.tsx b/torchci/components/metrics/panels/TimeSeriesPanel.tsx
@@ -223,7 +223,7 @@ export default function TimeSeriesPanel({
   yAxisLabel,
   // Additional EChartsOption (ex max y value)
   additionalOptions,
-  useClickHouse = false,
+  useClickHouse = true,
 }: {
   title: string;
   queryCollection?: string;

diff --git a/torchci/lib/fetchPR.ts b/torchci/lib/fetchPR.ts
@@ -51,7 +51,7 @@ export default async function fetchPR(
   repo: string,
   prNumber: string,
   octokit: Octokit,
-  useClickhouse: boolean = false
+  useClickhouse: boolean = true
 ): Promise<PRData> {
   // We pull data from both Rockset and Github to get all commits, including
   // the ones that have been force merged out of the git history.