chore: Add supoort for trino 479, remove support for 451 and 476 (#839)

NickLarsenNZ · web-flow · commit c0050c65e80b · 2026-01-26T11:31:23.000Z
* chore: Add supoort for trino 479, remove support for 451 and 476

* test: Add trino 479, remove 451 and 476

* docs: Update supported versions

* docs: Update getting started to use 479

* chore: Update changelog

* docs: Update trino to 479

* test: Update trino to 479

* docs: Update getting started to use trino-cli 479

* docs: Update trino-cli to 479
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@ All notable changes to this project will be documented in this file.
 - Support objectOverrides using `.spec.objectOverrides`.
   See [objectOverrides concepts page](https://docs.stackable.tech/home/nightly/concepts/overrides/#object-overrides) for details ([#831]).
 - Enable the [restart-controller](https://docs.stackable.tech/home/nightly/commons-operator/restarter/), so that the Pods are automatically restarted on config changes ([#833]).
+- Add support for Trino 479 ([#839]).
 - Add `enableColumnMasking` field to `opa` configuration in `authorization` ([#827]).
 - Support batched column masks in Rego rules ([#827]).
 
@@ -19,9 +20,14 @@ All notable changes to this project will be documented in this file.
 - BREAKING: The operator no longer sets `opa.policy.column-masking-uri` in `access-control.properties` but
   `opa.policy.batch-column-masking-uri` instead, allowing Trino to fetch multiple column masks in a single request ([#827]).
 
+### Removed
+
+- Remove support for Trino 451 and 476 ([#839]).
+
+[#827]: https://github.com/stackabletech/trino-operator/pull/827
 [#831]: https://github.com/stackabletech/trino-operator/pull/831
 [#833]: https://github.com/stackabletech/trino-operator/pull/833
-[#827]: https://github.com/stackabletech/trino-operator/pull/827
+[#839]: https://github.com/stackabletech/trino-operator/pull/839
 
 ## [25.11.0] - 2025-11-07
 
diff --git a/docs/modules/trino/examples/getting_started/code/getting_started.sh b/docs/modules/trino/examples/getting_started/code/getting_started.sh
@@ -77,7 +77,7 @@ sleep 5
 echo "Start testing Trino"
 echo "Downloading Trino CLI tool as trino.jar"
 # tag::download-trino-cli[]
-curl --fail --output trino.jar https://repo.stackable.tech/repository/packages/trino-cli/trino-cli-477
+curl --fail --output trino.jar https://repo.stackable.tech/repository/packages/trino-cli/trino-cli-479
 # end::download-trino-cli[]
 
 echo "Run chmod +x for trino.jar"
diff --git a/docs/modules/trino/examples/getting_started/code/getting_started.sh.j2 b/docs/modules/trino/examples/getting_started/code/getting_started.sh.j2
@@ -77,7 +77,7 @@ sleep 5
 echo "Start testing Trino"
 echo "Downloading Trino CLI tool as trino.jar"
 # tag::download-trino-cli[]
-curl --fail --output trino.jar https://repo.stackable.tech/repository/packages/trino-cli/trino-cli-477
+curl --fail --output trino.jar https://repo.stackable.tech/repository/packages/trino-cli/trino-cli-479
 # end::download-trino-cli[]
 
 echo "Run chmod +x for trino.jar"
diff --git a/docs/modules/trino/examples/getting_started/code/trino.yaml b/docs/modules/trino/examples/getting_started/code/trino.yaml
@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/docs/modules/trino/examples/getting_started/code/trino.yaml.j2 b/docs/modules/trino/examples/getting_started/code/trino.yaml.j2
@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/docs/modules/trino/examples/usage-guide/fault-tolerant-execution.yaml b/docs/modules/trino/examples/usage-guide/fault-tolerant-execution.yaml
@@ -5,7 +5,7 @@ metadata:
   name: trino-fault-tolerant
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/docs/modules/trino/examples/usage-guide/trino-insecure.yaml b/docs/modules/trino/examples/usage-guide/trino-insecure.yaml
@@ -17,7 +17,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/docs/modules/trino/examples/usage-guide/trino-secure-internal-tls.yaml b/docs/modules/trino/examples/usage-guide/trino-secure-internal-tls.yaml
@@ -17,7 +17,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     tls:
       internalSecretClass: trino-internal-tls # <1>
diff --git a/docs/modules/trino/examples/usage-guide/trino-secure-tls-only.yaml b/docs/modules/trino/examples/usage-guide/trino-secure-tls-only.yaml
@@ -17,7 +17,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     tls:
       serverSecretClass: trino-tls # <1>
diff --git a/docs/modules/trino/examples/usage-guide/trino-secure-tls.yaml b/docs/modules/trino/examples/usage-guide/trino-secure-tls.yaml
@@ -17,7 +17,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     tls:
       serverSecretClass: trino-tls # <1>
diff --git a/docs/modules/trino/pages/getting_started/installation.adoc b/docs/modules/trino/pages/getting_started/installation.adoc
@@ -57,7 +57,7 @@ For these components extra steps are required.
 ** a Stackable xref:secret-operator:index.adoc[Secret Operator] for certificates when deploying for TLS
 ** a Stackable xref:commons-operator:index.adoc[Commons Operator] for certificates when deploying for TLS authentication
 ** (for authorization): a Stackable xref:opa:index.adoc[OPA Operator]
-** the https://repo.stackable.tech/#browse/browse:packages:trino-cli%2Ftrino-cli-477[Trino CLI] to test SQL queries
+** the https://repo.stackable.tech/#browse/browse:packages:trino-cli%2Ftrino-cli-479[Trino CLI] to test SQL queries
 
 === S3 bucket
 
diff --git a/docs/modules/trino/pages/usage-guide/catalogs/index.adoc b/docs/modules/trino/pages/usage-guide/catalogs/index.adoc
@@ -93,7 +93,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/docs/modules/trino/pages/usage-guide/connect_to_trino.adoc b/docs/modules/trino/pages/usage-guide/connect_to_trino.adoc
@@ -62,7 +62,7 @@ The `--insecure` flag ignores the server TLS certificate and is required in this
 
 [source,console]
 ----
-$ trino-cli-477 --server https://85.215.195.29:8443 --user admin --password --insecure
+$ trino-cli-479 --server https://85.215.195.29:8443 --user admin --password --insecure
 ----
 
 TIP: In case you are using OpenID connect, use `--external-authentication` instead of `--password`.
diff --git a/docs/modules/trino/partials/supported-versions.adoc b/docs/modules/trino/partials/supported-versions.adoc
@@ -3,6 +3,5 @@
 // Stackable Platform documentation.
 
 // These versions must be sorted in descending order (highest to lowest).
+- 479
 - 477 (LTS)
-- 476 (deprecated)
-- 451 (deprecated)
diff --git a/examples/simple-trino-cluster-authentication-opa-authorization-s3.yaml b/examples/simple-trino-cluster-authentication-opa-authorization-s3.yaml
@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     authentication:
       - authenticationClass: simple-trino-users
diff --git a/examples/simple-trino-cluster-hive-ha-s3.yaml b/examples/simple-trino-cluster-hive-ha-s3.yaml
@@ -9,7 +9,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/examples/simple-trino-cluster-resource-limits.yaml b/examples/simple-trino-cluster-resource-limits.yaml
@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector: {}
   coordinators:
diff --git a/examples/simple-trino-cluster-s3.yaml b/examples/simple-trino-cluster-s3.yaml
@@ -7,7 +7,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/examples/simple-trino-cluster.yaml b/examples/simple-trino-cluster.yaml
@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:
diff --git a/examples/simple-trino-oauth2.yaml b/examples/simple-trino-oauth2.yaml
@@ -91,7 +91,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "477"
+    productVersion: "479"
   clusterConfig:
     authentication:
       - authenticationClass: simple-trino-oidc
diff --git a/rust/operator-binary/src/config/jvm.rs b/rust/operator-binary/src/config/jvm.rs
@@ -37,7 +37,7 @@ pub enum Error {
     MergeJvmArgumentOverrides { source: role_utils::Error },
 }
 
-// Currently works for all supported versions (451 and 455 as of 2024-09-04) but maybe be changed
+// Currently works for all supported versions (as of 2024-09-04) but maybe be changed
 // in the future depending on the role and version.
 pub fn jvm_config(
     product_version: u16,
@@ -110,28 +110,10 @@ fn recommended_trino_jvm_args(_product_version: u16) -> Result<Vec<String>, Erro
 #[cfg(not(test))]
 fn recommended_trino_jvm_args(product_version: u16) -> Result<Vec<String>, Error> {
     match product_version {
-        // Copied from https://trino.io/docs/451/installation/deployment.html
-        451 => Ok(vec![
-            "-XX:InitialRAMPercentage=80".to_owned(),
-            "-XX:MaxRAMPercentage=80".to_owned(),
-            "-XX:G1HeapRegionSize=32M".to_owned(),
-            "-XX:+ExitOnOutOfMemoryError".to_owned(),
-            "-XX:+HeapDumpOnOutOfMemoryError".to_owned(),
-            "-XX:-OmitStackTraceInFastThrow".to_owned(),
-            "-XX:ReservedCodeCacheSize=512M".to_owned(),
-            "-XX:PerMethodRecompilationCutoff=10000".to_owned(),
-            "-XX:PerBytecodeRecompilationCutoff=10000".to_owned(),
-            "-Djdk.attach.allowAttachSelf=true".to_owned(),
-            "-Djdk.nio.maxCachedBufferSize=2000000".to_owned(),
-            "-Dfile.encoding=UTF-8".to_owned(),
-            "-XX:+EnableDynamicAgentLoading".to_owned(),
-            "-XX:+UnlockDiagnosticVMOptions".to_owned(),
-            "-XX:G1NumCollectionsKeepPinned=10000000".to_owned(),
-        ]),
         // Copied from:
-        // - https://trino.io/docs/476/installation/deployment.html#jvm-config
         // - https://trino.io/docs/477/installation/deployment.html#jvm-config
-        476 | 477 => Ok(vec![
+        // - https://trino.io/docs/479/installation/deployment.html#jvm-config
+        477 | 479 => Ok(vec![
             "-XX:InitialRAMPercentage=80".to_owned(),
             "-XX:MaxRAMPercentage=80".to_owned(),
             "-XX:G1HeapRegionSize=32M".to_owned(),
@@ -172,7 +154,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
           coordinators:
@@ -216,7 +198,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
           coordinators:
diff --git a/rust/operator-binary/src/controller.rs b/rust/operator-binary/src/controller.rs
@@ -1887,7 +1887,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector:
               matchLabels:
@@ -1945,7 +1945,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector:
               matchLabels:
@@ -2135,7 +2135,7 @@ mod tests {
           name: trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector:
               matchLabels:
@@ -2188,7 +2188,7 @@ mod tests {
           name: trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector:
               matchLabels:
diff --git a/rust/operator-binary/src/crd/affinity.rs b/rust/operator-binary/src/crd/affinity.rs
@@ -115,7 +115,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector:
               matchLabels:
@@ -203,7 +203,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector:
               matchLabels:
diff --git a/rust/operator-binary/src/crd/mod.rs b/rust/operator-binary/src/crd/mod.rs
@@ -1043,7 +1043,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
         "#;
@@ -1059,7 +1059,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
             tls:
@@ -1077,7 +1077,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
             tls:
@@ -1096,7 +1096,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
             tls:
@@ -1117,7 +1117,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
         "#;
@@ -1133,7 +1133,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
             tls:
@@ -1151,7 +1151,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
             tls:
@@ -1173,7 +1173,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
         "#;
@@ -1194,7 +1194,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
           workers:
@@ -1221,7 +1221,7 @@ mod tests {
           name: simple-trino
         spec:
           image:
-            productVersion: "477"
+            productVersion: "479"
           clusterConfig:
             catalogLabelSelector: {}
           workers:
diff --git a/tests/test-definition.yaml b/tests/test-definition.yaml
