fix: split interval function should return 4xx (#6608)

afhassan · web-flow · commit 72f03ab2494f · 2025-02-25T22:04:44.000-08:00
Signed-off-by: Ahmed Hassan &lt;afayekhassan@gmail.com&gt;
diff --git a/pkg/querier/tripperware/queryrange/split_by_interval.go b/pkg/querier/tripperware/queryrange/split_by_interval.go
@@ -54,7 +54,7 @@ func (s splitByInterval) Do(ctx context.Context, r tripperware.Request) (tripper
 	// to line up the boundaries with step.
 	interval, err := s.interval(ctx, r)
 	if err != nil {
-		return nil, httpgrpc.Errorf(http.StatusInternalServerError, err.Error())
+		return nil, httpgrpc.Errorf(http.StatusBadRequest, err.Error())
 	}
 	reqs, err := splitQuery(r, interval)
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ func (s splitByInterval) Do(ctx context.Context, r tripperware.Request) (tripper`
`54`	`54`	`// to line up the boundaries with step.`
`55`	`55`	`interval, err := s.interval(ctx, r)`
`56`	`56`	`if err != nil {`
`57`		`- return nil, httpgrpc.Errorf(http.StatusInternalServerError, err.Error())`
	`57`	`+ return nil, httpgrpc.Errorf(http.StatusBadRequest, err.Error())`
`58`	`58`	`}`
`59`	`59`	`reqs, err := splitQuery(r, interval)`
`60`	`60`	`if err != nil {`