Skip to content

Commit 3941572

Browse files
authored
fix(db): drop tp/ep/num_gpu >= 1 checks to allow aggregated multinode configs (#299)
1 parent 258f7aa commit 3941572

1 file changed

Lines changed: 17 additions & 0 deletions

File tree

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
-- ============================================================
2+
-- CONFIGS — allow prefill/decode tp / ep / num_gpu = 0
3+
-- ============================================================
4+
--
5+
-- Aggregated multinode disagg runs can produce a config with no decode
6+
-- workers for low-conc sweeps (e.g. conc=1 on dsv4 run 25296668638 had
7+
-- decode_tp=0, decode_ep=0, num_decode_gpu=0 with all 8 GPUs on prefill).
8+
-- The original `>= 1` checks rejected these rows during ingestion, so the
9+
-- conc=1 point silently disappeared from the official chart. Drop the
10+
-- prefill-side mirrors too for symmetry.
11+
12+
alter table configs drop constraint configs_decode_tp_positive;
13+
alter table configs drop constraint configs_decode_ep_positive;
14+
alter table configs drop constraint configs_num_decode_gpu_positive;
15+
alter table configs drop constraint configs_prefill_tp_positive;
16+
alter table configs drop constraint configs_prefill_ep_positive;
17+
alter table configs drop constraint configs_num_prefill_gpu_positive;

0 commit comments

Comments
 (0)