Skip to content

Commit b0c5923

Browse files
committed
refactor: Remove unused ctx parameter from convertCountResponse
Removes the unused context parameter from the convertCountResponse method and updates its call site. The context was not being used within the function. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]> Signed-off-by: Andres Tobon <[email protected]>
1 parent 8114744 commit b0c5923

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

internal/infrastructure/opensearch/searcher.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ func (os *OpenSearchSearcher) QueryResourcesCount(
121121

122122
slog.DebugContext(ctx, "aggregation response", "response", aggregationResponse)
123123

124-
result, err := os.convertCountResponse(ctx, countResponse, aggregationResponse)
124+
result, err := os.convertCountResponse(countResponse, aggregationResponse)
125125
if err != nil {
126126
return nil, fmt.Errorf("failed to convert count response: %w", err)
127127
}
@@ -205,7 +205,7 @@ func (os *OpenSearchSearcher) convertHit(hit Hit) (model.Resource, error) {
205205
return resource, nil
206206
}
207207

208-
func (os *OpenSearchSearcher) convertCountResponse(ctx context.Context, response *CountResponse, aggregationResponse *AggregationResponse) (*model.CountResult, error) {
208+
func (os *OpenSearchSearcher) convertCountResponse(response *CountResponse, aggregationResponse *AggregationResponse) (*model.CountResult, error) {
209209
aggregation := model.TermsAggregation{
210210
DocCountErrorUpperBound: aggregationResponse.GroupBy.DocCountErrorUpperBound,
211211
SumOtherDocCount: aggregationResponse.GroupBy.SumOtherDocCount,

0 commit comments

Comments
 (0)