Refactored compatibility-verifier module

[abhioncbr]

Labels:

• refactor
• cleanup

Description:
In this PR, the focus is mainly on the cleanliness of the code in the pinot-compatibility-verifier module. Here is the summary

• Writing logs using logger instead of using the system.out
• leveraging the createSchema return value in table creation operation.
• Set the default character for the file objects.
• addressing particular minor warning issues.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.

Project coverage is 62.01%. Comparing base (59551e4) to head (c196986).
Report is 620 commits behind head on master.

Files	Patch %	Lines
...src/main/java/org/apache/pinot/compat/TableOp.java	0.00%	8 Missing ⚠️
...rg/apache/pinot/compat/CompatibilityOpsRunner.java	0.00%	5 Missing ⚠️
...rc/main/java/org/apache/pinot/compat/StreamOp.java	0.00%	4 Missing ⚠️
.../src/main/java/org/apache/pinot/compat/BaseOp.java	0.00%	2 Missing ⚠️
...ava/org/apache/pinot/compat/ClusterDescriptor.java	0.00%	1 Missing ⚠️
...src/main/java/org/apache/pinot/compat/QueryOp.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #13359      +/-   ##
============================================
+ Coverage     61.75%   62.01%   +0.26%     
+ Complexity      207      198       -9     
============================================
  Files          2436     2550     +114     
  Lines        133233   140256    +7023     
  Branches      20636    21802    +1166     
============================================
+ Hits          82274    86976    +4702     
- Misses        44911    46688    +1777     
- Partials       6048     6592     +544     

Flag	Coverage Δ
custom-integration1	<0.01% <ø> (-0.01%)	⬇️
integration	<0.01% <ø> (-0.01%)	⬇️
integration1	<0.01% <ø> (-0.01%)	⬇️
integration2	0.00% <ø> (ø)
java-11	61.95% <0.00%> (+0.24%)	⬆️
java-21	61.90% <0.00%> (+0.27%)	⬆️
skip-bytebuffers-false	61.98% <0.00%> (+0.24%)	⬆️
skip-bytebuffers-true	61.86% <0.00%> (+34.13%)	⬆️
temurin	62.01% <0.00%> (+0.26%)	⬆️
unittests	62.00% <0.00%> (+0.26%)	⬆️
unittests1	46.58% <ø> (-0.31%)	⬇️
unittests2	27.66% <0.00%> (-0.07%)	⬇️

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.

[Jackie-Jiang]

I took a look at the compatibility test run, and didn't see any log from BaseOp. Is this expected?