[Presto pinot driver] Adding NoopPinotMetricFactory and corresponding changes

[xiangfu0]

Description

Create NoopPinotMetricFactory for PinotScatterGatherQueryClient to bypass the requirement for pinot-yammer runtime dependency.

Upgrade Notes

Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

• Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

• Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

• New configuration options
• Deprecation of configurations
• Signature changes to public methods/interfaces
• New plugins added or old plugins removed

• Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

[codecov-commenter]

Codecov Report

Merging #8270 (775ac8a) into master (1c4512e) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #8270      +/-   ##
============================================
- Coverage     70.81%   70.74%   -0.08%     
+ Complexity     4244     4241       -3     
============================================
  Files          1631     1631              
  Lines         85454    85454              
  Branches      12877    12877              
============================================
- Hits          60515    60455      -60     
- Misses        20758    20824      +66     
+ Partials       4181     4175       -6     

Flag	Coverage Δ
integration1	28.91% <ø> (+<0.01%)	⬆️
integration2	27.58% <ø> (-0.10%)	⬇️
unittests1	66.92% <ø> (-0.05%)	⬇️
unittests2	14.16% <ø> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
...data/manager/realtime/DefaultSegmentCommitter.java	0.00% <0.00%> (-80.00%)	⬇️
...a/manager/realtime/RealtimeSegmentDataManager.java	50.00% <0.00%> (-25.00%)	⬇️
...ller/helix/core/minion/TaskTypeMetricsUpdater.java	80.00% <0.00%> (-20.00%)	⬇️
...er/api/resources/LLCSegmentCompletionHandlers.java	43.56% <0.00%> (-18.82%)	⬇️
...nt/local/startree/v2/store/StarTreeDataSource.java	40.00% <0.00%> (-13.34%)	⬇️
...data/manager/realtime/SegmentCommitterFactory.java	88.23% <0.00%> (-11.77%)	⬇️
...altime/ServerSegmentCompletionProtocolHandler.java	51.42% <0.00%> (-6.67%)	⬇️
...core/startree/operator/StarTreeFilterOperator.java	87.41% <0.00%> (-5.60%)	⬇️
...ot/segment/local/startree/OffHeapStarTreeNode.java	72.22% <0.00%> (-5.56%)	⬇️
...pache/pinot/core/query/optimizer/filter/Range.java	95.91% <0.00%> (-4.09%)	⬇️
... and 21 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 1c4512e...775ac8a. Read the comment docs.