revert

iamrajiv · iamrajiv · commit eb2b7b77d94e · 2025-11-21T14:50:36.000+05:30
diff --git a/modules/frontend/search_sharder.go b/modules/frontend/search_sharder.go
@@ -105,7 +105,7 @@ func (s asyncSearchSharder) RoundTrip(pipelineRequest pipeline.Request) (pipelin
 	// Validate SpansPerSpanSet against MaxSpansPerSpanSet
 	// If MaxSpansPerSpanSet is 0, it means unlimited spans are allowed
 	// If MaxSpansPerSpanSet is non-zero, enforce the limit
-	if s.cfg.MaxSpansPerSpanSet >= 0 && searchReq.SpansPerSpanSet > s.cfg.MaxSpansPerSpanSet {
+	if s.cfg.MaxSpansPerSpanSet != 0 && searchReq.SpansPerSpanSet > s.cfg.MaxSpansPerSpanSet {
 		return pipeline.NewBadRequest(fmt.Errorf("spans per span set exceeds %d. received %d", s.cfg.MaxSpansPerSpanSet, searchReq.SpansPerSpanSet)), nil
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ func (s asyncSearchSharder) RoundTrip(pipelineRequest pipeline.Request) (pipelin`
`105`	`105`	`// Validate SpansPerSpanSet against MaxSpansPerSpanSet`
`106`	`106`	`// If MaxSpansPerSpanSet is 0, it means unlimited spans are allowed`
`107`	`107`	`// If MaxSpansPerSpanSet is non-zero, enforce the limit`
`108`		`- if s.cfg.MaxSpansPerSpanSet >= 0 && searchReq.SpansPerSpanSet > s.cfg.MaxSpansPerSpanSet {`
	`108`	`+ if s.cfg.MaxSpansPerSpanSet != 0 && searchReq.SpansPerSpanSet > s.cfg.MaxSpansPerSpanSet {`
`109`	`109`	`return pipeline.NewBadRequest(fmt.Errorf("spans per span set exceeds %d. received %d", s.cfg.MaxSpansPerSpanSet, searchReq.SpansPerSpanSet)), nil`
`110`	`110`	`}`
`111`	`111`