From 96f7e716910462556fc5c365e6769302ee92e987 Mon Sep 17 00:00:00 2001
From: mikebender <mikebender@deephaven.io>
Date: Wed, 22 Nov 2023 10:10:41 -0500
Subject: [PATCH] fix: Add BigDecimal and BigInteger as numeric aggregation
 options

- BigDecimal and BigInteger weren't listed as numeric types, and were not getting aggregated properly
- Fixes #4877
---
 .../web/client/api/tree/enums/JsAggregationOperation.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/web/client-api/src/main/java/io/deephaven/web/client/api/tree/enums/JsAggregationOperation.java b/web/client-api/src/main/java/io/deephaven/web/client/api/tree/enums/JsAggregationOperation.java
index acd460e0859..79fa9a9ec5f 100644
--- a/web/client-api/src/main/java/io/deephaven/web/client/api/tree/enums/JsAggregationOperation.java
+++ b/web/client-api/src/main/java/io/deephaven/web/client/api/tree/enums/JsAggregationOperation.java
@@ -128,7 +128,9 @@ private static boolean isNumeric(String columnType) {
             case "long":
             case "short":
             case "char":
-            case "byte": {
+            case "byte":
+            case "java.math.BigDecimal":
+            case "java.math.BigInteger": {
                 return true;
             }
         }