ifNotFinite fix

Author: matt-aitken

internal-packages/clickhouse/src/tsqlFunctions.test.ts

@@ -692,9 +692,7 @@ describe("TSQL Function Smoke Tests", () => {
     await runCases(client, [
       ["isFinite", "SELECT isFinite(1.0) AS r FROM task_runs"],
       ["isInfinite", "SELECT isInfinite(1.0 / 0) AS r FROM task_runs"],
-      // ifNotFinite: TSQL definition has maxArgs: 1, but ClickHouse expects 2.
-      // Skipped until the function definition is fixed.
-      // ["ifNotFinite", "SELECT ifNotFinite(1.0 / 0, 0) AS r FROM task_runs"],
+      ["ifNotFinite", "SELECT ifNotFinite(1.0 / 0, 0) AS r FROM task_runs"],
       ["isNaN", "SELECT isNaN(0.0 / 0) AS r FROM task_runs"],
       ["bar", "SELECT bar(usage_duration_ms, 0, 10000, 20) AS r FROM task_runs"],
       [

internal-packages/tsql/src/query/functions.ts

@@ -445,7 +445,7 @@ export const TSQL_CLICKHOUSE_FUNCTIONS: Record<string, TSQLFunctionMeta> = {
   // Other functions
   isFinite: { clickhouseName: "isFinite", minArgs: 1, maxArgs: 1 },
   isInfinite: { clickhouseName: "isInfinite", minArgs: 1, maxArgs: 1 },
-  ifNotFinite: { clickhouseName: "ifNotFinite", minArgs: 1, maxArgs: 1 },
+  ifNotFinite: { clickhouseName: "ifNotFinite", minArgs: 2, maxArgs: 2 },
   isNaN: { clickhouseName: "isNaN", minArgs: 1, maxArgs: 1 },
   bar: { clickhouseName: "bar", minArgs: 4, maxArgs: 4 },
   transform: { clickhouseName: "transform", minArgs: 3, maxArgs: 4 },