Fix default value for additionalJVMConfig

`additionalJVMConfig` expects a list, so the default value should not be an empty object.
trinodb · May 10, 2024 · 3239a09 · 3239a09
1 parent a3a62a1
commit 3239a09
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/charts/trino/README.md b/charts/trino/README.md
@@ -66,7 +66,7 @@ The following table lists the configurable parameters of the Trino chart and the
 | `coordinator.jvm.gcMethod.g1.heapRegionSize` |  | `"32M"` |
 | `coordinator.config.memory.heapHeadroomPerNode` |  | `""` |
 | `coordinator.config.query.maxMemoryPerNode` |  | `"1GB"` |
-| `coordinator.additionalJVMConfig` |  | `{}` |
+| `coordinator.additionalJVMConfig` |  | `[]` |
 | `coordinator.additionalExposedPorts` |  | `{}` |
 | `coordinator.resources` |  | `{}` |
 | `coordinator.livenessProbe` |  | `{}` |
@@ -85,7 +85,7 @@ The following table lists the configurable parameters of the Trino chart and the
 | `worker.jvm.gcMethod.g1.heapRegionSize` |  | `"32M"` |
 | `worker.config.memory.heapHeadroomPerNode` |  | `""` |
 | `worker.config.query.maxMemoryPerNode` |  | `"1GB"` |
-| `worker.additionalJVMConfig` |  | `{}` |
+| `worker.additionalJVMConfig` |  | `[]` |
 | `worker.additionalExposedPorts` |  | `{}` |
 | `worker.resources` |  | `{}` |
 | `worker.livenessProbe` |  | `{}` |

diff --git a/charts/trino/values.yaml b/charts/trino/values.yaml
@@ -266,7 +266,7 @@ coordinator:
     query:
       maxMemoryPerNode: "1GB"
 
-  additionalJVMConfig: {}
+  additionalJVMConfig: []
 
   additionalExposedPorts: {}
 
@@ -335,7 +335,7 @@ worker:
     query:
       maxMemoryPerNode: "1GB"
 
-  additionalJVMConfig: {}
+  additionalJVMConfig: []
 
   additionalExposedPorts: {}