shauryachats opened a new pull request, #17440:
URL: https://github.com/apache/pinot/pull/17440

   ## Summary
   
   This PR enhances the time series query execution framework to properly 
propagate and expose query exceptions instead of throwing them, enabling better 
error handling and reporting for both broker-compatible and 
Prometheus-compatible APIs.
   This enables partial results with warnings instead of complete failures, as 
well as exceptions from leaf operators visible at the broker level.
   
   The exceptions are encoded in the `metadataMap` passed alongside data, and 
stored in `TimeSeriesBlock` for upward propagation.
   
   To support partial results for Prometheus-compatible API, this PR also 
introduces a `warnings` field (as specified in the [Prometheus API 
reference](https://prometheus.io/docs/prometheus/latest/querying/api/)) to 
expose these warnings correctly.
   
   ## Testing
   Added integration test `testQueryWithServerException` to verify exception 
handling for both response formats:
     - Broker-compatible API: validates exceptions are properly returned with 
error codes.
     - Prometheus-compatible API: validates warnings are included in success 
responses.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to