Skip to content

Commit

Permalink
Revert "Check for errors when iterating through chunks or samples in …
Browse files Browse the repository at this point in the history
…response to a query (#6451)" (#6576)

This reverts commit 83d0d7b.
  • Loading branch information
pstibrany authored Nov 6, 2023
1 parent 890d1d3 commit ad50cda
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 15 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
* [BUGFIX] Querier: reduce log volume when querying ingesters with zone-awareness enabled and one or more instances in a single zone unavailable. #6381
* [BUGFIX] Querier: don't try to query further ingesters if ingester query request minimization is enabled and a query limit is reached as a result of the responses from the initial set of ingesters. #6402
* [BUGFIX] Ingester: Don't cache context cancellation error when querying. #6446
* [BUGFIX] Ingester: don't ignore errors encountered while iterating through chunks or samples in response to a query request. #6451 #6469
* [BUGFIX] Ingester: don't ignore errors encountered while iterating through chunks or samples in response to a query request. #6469
* [BUGFIX] All: fix issue where traces for some inter-component gRPC calls would incorrectly show the call as failing due to cancellation. #6470
* [BUGFIX] Querier: correctly mark streaming requests to ingesters or store-gateways as successful, not cancelled, in metrics and traces. #6471 #6505

Expand Down
14 changes: 0 additions & 14 deletions pkg/ingester/ingester.go
Original file line number Diff line number Diff line change
Expand Up @@ -1769,11 +1769,6 @@ func (i *Ingester) executeSamplesQuery(ctx context.Context, db *userTSDB, from,
return 0, 0, fmt.Errorf("unsupported value type: %v", valType)
}
}

if err := it.Err(); err != nil {
return 0, 0, err
}

numSamples += len(ts.Samples) + len(ts.Histograms)
numSeries++
tsSize := ts.Size()
Expand Down Expand Up @@ -1870,11 +1865,6 @@ func (i *Ingester) executeChunksQuery(ctx context.Context, db *userTSDB, from, t
ts.Chunks = append(ts.Chunks, ch)
numSamples += meta.Chunk.NumSamples()
}

if err := it.Err(); err != nil {
return 0, 0, err
}

numSeries++
tsSize := ts.Size()

Expand Down Expand Up @@ -2094,10 +2084,6 @@ func (i *Ingester) sendStreamingQueryChunks(allSeries *chunkSeriesNode, stream c
numSamples += meta.Chunk.NumSamples()
}

if err := it.Err(); err != nil {
return 0, 0, 0, err
}

numChunks += len(seriesChunks.Chunks)
msgSize := seriesChunks.Size()

Expand Down

0 comments on commit ad50cda

Please sign in to comment.