bubble up error messages from broker

[agavra]

This PR improves error message handling on the broker and ensures that root causes are returned to the user. There's probably a lot of additional work to be done to make sure that the response is improved for all different types of error messages, but this is an improvement over the status quo.

cc @npawar

Local Testing:

[codecov-commenter]

Codecov Report

Merging #9754 (b1662bf) into master (2f640ff) will decrease coverage by 1.57%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #9754      +/-   ##
============================================
- Coverage     70.08%   68.50%   -1.58%     
+ Complexity     4980     4967      -13     
============================================
  Files          1951     1953       +2     
  Lines        104561   104801     +240     
  Branches      15836    15871      +35     
============================================
- Hits          73279    71794    -1485     
- Misses        26155    27889    +1734     
+ Partials       5127     5118       -9     

Flag	Coverage Δ
integration1	25.30% <66.66%> (-0.14%)	⬇️
integration2	?
unittests1	67.57% <ø> (+0.01%)	⬆️
unittests2	15.64% <0.00%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...t/controller/api/resources/PinotQueryResource.java	11.18% <0.00%> (-43.04%)	⬇️
...pinot/broker/api/resources/PinotClientRequest.java	45.71% <100.00%> (+2.43%)	⬆️
...pinot/core/data/manager/realtime/TimerService.java	0.00% <0.00%> (-100.00%)	⬇️
...t/core/plan/StreamingInstanceResponsePlanNode.java	0.00% <0.00%> (-100.00%)	⬇️
...ore/operator/streaming/StreamingResponseUtils.java	0.00% <0.00%> (-100.00%)	⬇️
...server/starter/helix/SegmentReloadStatusValue.java	0.00% <0.00%> (-100.00%)	⬇️
...ager/realtime/PeerSchemeSplitSegmentCommitter.java	0.00% <0.00%> (-100.00%)	⬇️
...urces/ServerReloadControllerJobStatusResponse.java	0.00% <0.00%> (-100.00%)	⬇️
...he/pinot/common/utils/grpc/GrpcRequestBuilder.java	0.00% <0.00%> (-90.91%)	⬇️
...ator/streaming/StreamingSelectionOnlyOperator.java	0.00% <0.00%> (-90.00%)	⬇️
... and 164 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[walterddr]

curious why is this example an exception? this seems like a bug to me. was it thrown cuz the int/double matching issue; or the agg field is not escaped?

[agavra]

@walterddr I agree with you, but that's orthogonal to this PR 😉