From fa233eefbd57593344cc35649ef0ab525205fa4b Mon Sep 17 00:00:00 2001
From: Mike Bender <mikebender@deephaven.io>
Date: Mon, 4 Dec 2023 16:34:17 -0500
Subject: [PATCH] fix: Add BigDecimal and BigInteger as numeric aggregation
 options (#4878)

- 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;
             }
         }