Skip to content

Commit 1f2df71

Browse files
jasonhernandezdef-
andcommitted
fix(materialize): improve CI test stability and compatibility
This commit addresses several issues to improve Materialize test reliability: - Update to Materialize v26 syntax: Replace deprecated COUNTER load generator with AUCTION, update to FOR ALL TABLES syntax for multi-output sources - Fix test parallelization: Serialize array, source, and subscribe tests to avoid resource contention and subsource name collisions - Optimize data loading: Cache list_tables() result during CSV loading to reduce expensive database calls - Fix doctest failures: Reduce to 54 doctest +SKIP markers matching postgres backend pattern (68% reduction) - Fix dead fixture warnings: Mark TPC benchmark tests as skipped instead of deselecting them to prevent pytest-deadfixtures from flagging their fixtures - Fix test_map: Make assertion Materialize-specific to avoid breaking other backends' pytest diff output - Remove deprecated enable_load_generator_counter and enable_create_table_from_source flags from compose.yaml Co-Authored-By: Dennis Felsing <[email protected]>
1 parent 956c1f9 commit 1f2df71

File tree

7 files changed

+317
-766
lines changed

7 files changed

+317
-766
lines changed

.github/workflows/ibis-backends.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,6 @@ jobs:
168168
- libpq5
169169
- name: materialize
170170
title: Materialize
171-
serial: true
172171
services:
173172
- materialize
174173
extras:
@@ -346,7 +345,6 @@ jobs:
346345
backend:
347346
name: materialize
348347
title: Materialize
349-
serial: true
350348
services:
351349
- materialize
352350
extras:

compose.yaml

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -557,24 +557,19 @@ services:
557557
- impala
558558

559559
materialize:
560-
image: materialize/materialized:v0.160.0
560+
image: materialize/materialized:v26.0.0
561561
ports:
562562
- 6875:6875 # SQL port
563563
- 6876:6876 # HTTP port
564564
- 6874:6874 # Internal port
565565
environment:
566566
MZ_LOG: info
567567
MZ_TELEMETRY: "false"
568-
MZ_SYSTEM_PARAMETER_DEFAULT: "max_tables=1000" # Increase from default 200 for testing
568+
MZ_SYSTEM_PARAMETER_DEFAULT: "max_tables=1000"
569569
healthcheck:
570-
test:
571-
- CMD-SHELL
572-
- bash -c 'printf "GET / HTTP/1.1\n\n" > /dev/tcp/127.0.0.1/6875; exit $$?;'
573-
- bash -c 'printf "GET / HTTP/1.1\n\n" > /dev/tcp/127.0.0.1:6876; exit $$?;'
574-
- bash -c 'printf "GET / HTTP/1.1\n\n" > /dev/tcp/127.0.0.1:6874; exit $$?;'
575-
interval: 2s
576-
retries: 30
577-
timeout: 5s
570+
test: ["CMD", "curl", "-f", "localhost:6878/api/readyz"]
571+
interval: 1s
572+
start_period: 60s
578573
volumes:
579574
- materialize:/data
580575
networks:

0 commit comments

Comments
 (0)