Skip to content

Commit da557fd

Browse files
razvansbernauer
andauthored
Apply suggestions from code review
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
1 parent 9e55ce1 commit da557fd

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

rust/operator-binary/src/controller.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ pub enum Error {
133133
#[snafu(display("object defines no namespace"))]
134134
ObjectHasNoNamespace,
135135

136-
#[snafu(display("Trino cluster {} has no namespace", name))]
136+
#[snafu(display("Trino cluster {name:?} has no namespace"))]
137137
MissingTrinoNamespace {
138138
source: crate::crd::Error,
139139
name: String,

tests/test-definition.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,8 @@ tests:
117117
- openshift
118118
- name: client-spooling
119119
dimensions:
120+
# The client spooling protocol was introduced in Trino 466 but it only works reliably starting with Trino 476.
121+
# Long term we should test all versions
120122
- trino-latest
121123
- openshift
122124
- name: listener

0 commit comments

Comments
 (0)