Skip to content

Commit 4aa16e4

Browse files
authored
update broken tests due to single-column PARTITION BY / ORDER BY change in latest CH (#4298)
Fix [e2e test failures](https://github.com/PeerDB-io/peerdb/actions/runs/25718948064/job/75518084196): ClickHouse 26.5+ preserves the parens around single-column PARTITION BY / ORDER BY in system.tables; older versions unwrap them.
1 parent 1ed5a42 commit 4aa16e4

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

flow/e2e/clickhouse_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3025,8 +3025,9 @@ func (s ClickHouseSuite) Test_PartitionBy() {
30253025
)).Scan(&partitionKey, &sortingKey),
30263026
)
30273027
require.NoError(s.t, ch.Close())
3028-
require.Equal(s.t, "num", partitionKey)
3029-
require.Equal(s.t, "val", sortingKey)
3028+
// ClickHouse 26.5+ preserves the parens around single-column PARTITION BY / ORDER BY in system.tables; older versions unwrap them.
3029+
require.Contains(s.t, []string{"num", "(num)"}, partitionKey)
3030+
require.Contains(s.t, []string{"val", "(val)"}, sortingKey)
30303031

30313032
env.Cancel(s.t.Context())
30323033
RequireEnvCanceled(s.t, env)

0 commit comments

Comments
 (0)