Azure / Azure/data-api-builder

[Bug]: `max-query-parallelism` accepts values outside documented 1-8 range

Offen Anfängerfreundlich
#3,570 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
2.x health-endpoint
Vorherrschende Sprache
C#
Sterne
1.5k
Forks
370
Ø Merge
3 T. 22 Std.
Gemergte PRs (30 T.)
9

Beschreibung

The docs state the valid range for `max-query-parallelism` is 1-8. However, both `0` and `9` are accepted without error.

## Expected

DAB should reject values outside the 1-8 range at config deserialization.

## Actual

| Value | Result |
|-------|--------|
| `0` | Accepted, DAB starts, health checks run (9 checks returned) |
| `9` | Accepted, DAB starts, health checks run (9 checks returned) |

**Impact:** `max-query-parallelism=0` is particularly concerning — it's unclear whether this means "no parallelism" (sequential) or "unlimited parallelism". Testing showed health checks still execute and return results, so `0` likely defaults to some internal value.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne bei der Konfigurationsdeserialisierung von DAB für `max-query-parallelism` und verfolge, wo dessen dokumentierter Wertebereich dargestellt wird. Füge Abdeckung für die Werte 0 und 9 hinzu und überprüfe anschließend, dass Werte außerhalb des Bereichs abgelehnt werden, während Werte von 1 bis 8 weiterhin akzeptiert werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp
Bereich
api, backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Ruhig
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
68/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.