search

apache / pinot

Apache Pinot - A realtime distributed OLAP datastore
https://pinot.apache.org/
Apache License 2.0
5.39k stars 1.26k forks source link

Reload API has a difficult response #11588

Open jadami10 opened 12 months ago

jadami10 commented 12 months ago

The response from the reload API, https://github.com/apache/pinot/blob/011c072196bc7959155a34f4ae9877abd676669f/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java#L785, is pretty weirdly written. It's a string followed by encoded json. This should probably look like the rebalance API result where it's just JSON.

This would be a breaking change, but maybe worth it? cc @saurabhd336

ArnavBalyan commented 12 months ago

Hi please assign this to me

ArnavBalyan commented 11 months ago

Thanks for raising this, have raised a PR for fixing the response