perf: add COUNTING_OPS_INDEX for O(1) lookup (#7744)

vega · Oct 8, 2021 · a45b023 · a45b023
1 parent 36f29c7
commit a45b023
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/aggregate.ts b/src/aggregate.ts
@@ -62,9 +62,10 @@ export function isAggregateOp(a: string | ArgminDef | ArgmaxDef): a is Aggregate
 }
 
 export const COUNTING_OPS: NonArgAggregateOp[] = ['count', 'valid', 'missing', 'distinct'];
+export const COUNTING_OPS_INDEX = toSet(COUNTING_OPS);
 
 export function isCountingAggregateOp(aggregate?: string | Aggregate): boolean {
-  return isString(aggregate) && contains(COUNTING_OPS, aggregate);
+  return isString(aggregate) && COUNTING_OPS_INDEX[aggregate];
 }
 
 export function isMinMaxOp(aggregate?: Aggregate | string): boolean {