feat: add config to disableSimdJson (#154)

langchain-ai · Sep 12, 2024 · 037212c · 037212c
1 parent 262f051
commit 037212c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 2 deletions.
diff --git a/charts/langsmith/Chart.yaml b/charts/langsmith/Chart.yaml
@@ -5,5 +5,5 @@ maintainers:
     email: [email protected]
 description: Helm chart to deploy the langsmith application and all services it depends on.
 type: application
-version: 0.7.5
+version: 0.7.6
 appVersion: "0.7.20"
diff --git a/charts/langsmith/README.md b/charts/langsmith/README.md
@@ -1,6 +1,6 @@
 # langsmith
 
-![Version: 0.7.5](https://img.shields.io/badge/Version-0.7.5-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.7.20](https://img.shields.io/badge/AppVersion-0.7.20-informational?style=flat-square)
+![Version: 0.7.6](https://img.shields.io/badge/Version-0.7.6-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.7.20](https://img.shields.io/badge/AppVersion-0.7.20-informational?style=flat-square)
 
 Helm chart to deploy the langsmith application and all services it depends on.
 
@@ -19,6 +19,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | apiIngress.labels | object | `{}` |  |
 | apiIngress.subdomain | string | `""` |  |
 | apiIngress.tls | list | `[]` |  |
+| clickhouse.config.allowSimdjson | bool | `true` |  |
 | clickhouse.containerHttpPort | int | `8123` |  |
 | clickhouse.containerNativePort | int | `9000` |  |
 | clickhouse.external.database | string | `"default"` |  |
@@ -258,6 +259,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| clickhouse.config.allowSimdjson | bool | `true` |  |
 | clickhouse.containerHttpPort | int | `8123` |  |
 | clickhouse.containerNativePort | int | `9000` |  |
 | clickhouse.external.database | string | `"default"` |  |

diff --git a/charts/langsmith/templates/clickhouse/config-map.yaml b/charts/langsmith/templates/clickhouse/config-map.yaml
@@ -27,6 +27,9 @@ data:
                 <parallel_view_processing>1</parallel_view_processing>
                 <materialize_ttl_after_modify>0</materialize_ttl_after_modify>
                 <wait_for_async_insert_timeout>25</wait_for_async_insert_timeout>
+                {{- if not .Values.clickhouse.config.allowSimdjson }}
+                <allow_simdjson>0</allow_simdjson>
+                {{- end }}
             </default>
         </profiles>
     </clickhouse>

diff --git a/charts/langsmith/values.yaml b/charts/langsmith/values.yaml
@@ -266,6 +266,8 @@ backend:
 
 clickhouse:
   name: "clickhouse"
+  config:
+    allowSimdjson: true
   external:
     # If enabled, use the following values to connect to an external database. This will also disable the
     # creation of a clickhouse stateful-set and service.