Make TableDataManagerProvider pluggable

[Jackie-Jiang]

• Make TableDataManagerProvider pluggable similar to InstanceDataManager
• Cleanup CommonConstants and HelixInstanceDataManagerConfig to reduce duplications

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 35.29412% with 11 lines in your changes missing coverage. Please review.

Project coverage is 63.79%. Comparing base (59551e4) to head (2c7ee2a).
Report is 1342 commits behind head on master.

Files with missing lines	Patch %	Lines
.../starter/helix/HelixInstanceDataManagerConfig.java	0.00%	6 Missing ⚠️
...server/starter/helix/HelixInstanceDataManager.java	0.00%	4 Missing ⚠️
.../java/org/apache/pinot/server/conf/ServerConf.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #14470      +/-   ##
============================================
+ Coverage     61.75%   63.79%   +2.04%     
- Complexity      207     1566    +1359     
============================================
  Files          2436     2664     +228     
  Lines        133233   146216   +12983     
  Branches      20636    22403    +1767     
============================================
+ Hits          82274    93280   +11006     
- Misses        44911    46044    +1133     
- Partials       6048     6892     +844     

Flag	Coverage Δ
custom-integration1	100.00% <ø> (+99.99%)	⬆️
integration	100.00% <ø> (+99.99%)	⬆️
integration1	100.00% <ø> (+99.99%)	⬆️
integration2	0.00% <ø> (ø)
java-11	63.75% <35.29%> (+2.04%)	⬆️
java-21	63.68% <35.29%> (+2.06%)	⬆️
skip-bytebuffers-false	63.77% <35.29%> (+2.02%)	⬆️
skip-bytebuffers-true	63.66% <35.29%> (+35.94%)	⬆️
temurin	63.79% <35.29%> (+2.04%)	⬆️
unittests	63.79% <35.29%> (+2.04%)	⬆️
unittests1	55.58% <100.00%> (+8.69%)	⬆️
unittests2	34.11% <5.88%> (+6.37%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[KKcorps]

If someone wants to change only the realtime or offline table data manager, they'll still need to initialise the other one. How do we avoid this?

[Jackie-Jiang]

@KKcorps That's why we are making the data manager provider pluggable, and people can choose any data manager to override