Fix the group-by reduce handling when query times out

[Jackie-Jiang]

Fix #8449

When broker times out, there are multiple places not handled properly:

• Should check the return value of CountDownLatch.await() to identify if the query times out
• Future[] length should be the reduce group size
• When the query times out, the result table won't be set, so reading rows from the result table can cause NPE

[codecov-commenter]

Codecov Report

Merging #8450 (4c7349c) into master (91c9ce4) will increase coverage by 6.49%.
The diff coverage is 86.95%.

@@             Coverage Diff              @@
##             master    #8450      +/-   ##
============================================
+ Coverage     64.21%   70.70%   +6.49%     
- Complexity     4282     4283       +1     
============================================
  Files          1618     1663      +45     
  Lines         85376    87287    +1911     
  Branches      13004    13210     +206     
============================================
+ Hits          54820    61716    +6896     
+ Misses        26576    21281    -5295     
- Partials       3980     4290     +310     

Flag	Coverage Δ
integration1	27.25% <69.56%> (?)
integration2	25.95% <69.56%> (?)
unittests1	67.03% <73.91%> (-0.02%)	⬇️
unittests2	14.14% <0.00%> (-0.02%)	⬇️

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

Impacted Files	Coverage Δ
...not/core/query/reduce/GroupByDataTableReducer.java	90.55% <86.95%> (+4.03%)	⬆️
.../impl/dictionary/BaseOffHeapMutableDictionary.java	81.33% <0.00%> (-3.34%)	⬇️
.../aggregation/function/ModeAggregationFunction.java	86.48% <0.00%> (-2.17%)	⬇️
...org/apache/pinot/core/data/table/TableResizer.java	91.30% <0.00%> (-1.09%)	⬇️
...t/server/api/resources/DefaultExceptionMapper.java	0.00% <0.00%> (ø)
...g/apache/pinot/server/api/resources/ErrorInfo.java	0.00% <0.00%> (ø)
...ot/plugin/metrics/yammer/YammerMetricsFactory.java	100.00% <0.00%> (ø)
...ache/pinot/server/api/resources/ErrorResponse.java	0.00% <0.00%> (ø)
...rg/apache/pinot/server/starter/ServerInstance.java	88.99% <0.00%> (ø)
...gin/inputformat/avro/SimpleAvroMessageDecoder.java	0.00% <0.00%> (ø)
... and 371 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91c9ce4...4c7349c. Read the comment docs.

[ashishkf]

brokerRequest param is duplicated.

[Jackie-Jiang]

This is intentional. The first arg is the broker side broker request, second arg is the server side broker request.