Moving deleteSegment call from POST to DELETE call

[tibrewalpratik17]

label:
bugfix

Moving deleteSegments call from POST api to DELETE api like other delete-segment based apis.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 33.33333% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 61.38%. Comparing base (59551e4) to head (4d2a7cb).
Report is 150 commits behind head on master.

Files	Patch %	Lines
...ler/api/resources/PinotSegmentRestletResource.java	33.33%	5 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #12663      +/-   ##
============================================
- Coverage     61.75%   61.38%   -0.37%     
+ Complexity      207      198       -9     
============================================
  Files          2436     2452      +16     
  Lines        133233   133877     +644     
  Branches      20636    20733      +97     
============================================
- Hits          82274    82177      -97     
- Misses        44911    45533     +622     
- Partials       6048     6167     +119     

Flag	Coverage Δ
custom-integration1	?
integration	<0.01% <0.00%> (-0.01%)	⬇️
integration1	<0.01% <0.00%> (-0.01%)	⬇️
integration2	0.00% <0.00%> (ø)
java-11	61.37% <33.33%> (-0.34%)	⬇️
java-21	27.72% <33.33%> (-33.90%)	⬇️
skip-bytebuffers-false	61.38% <33.33%> (-0.37%)	⬇️
skip-bytebuffers-true	0.00% <0.00%> (-27.73%)	⬇️
temurin	61.38% <33.33%> (-0.37%)	⬇️
unittests	61.37% <33.33%> (-0.37%)	⬇️
unittests1	46.27% <ø> (-0.62%)	⬇️
unittests2	27.73% <33.33%> (+<0.01%)	⬆️

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]

Shall we integrate it into DELETE /segments/{tableName} and add an extra parameter for target segments? If segments are not provided, we delete all segments

[tibrewalpratik17]

Shall we integrate it into DELETE /segments/{tableName} and add an extra parameter for target segments? If segments are not provided, we delete all segments

Sounds good to me! Updated accordingly!

[Jackie-Jiang]

Suggest adding a test in PinotSegmentRestletResourceTest for the new usage

[tibrewalpratik17]

Suggest adding a test in PinotSegmentRestletResourceTest for the new usage

Will do Jackie.