somandal commented on PR #15891: URL: https://github.com/apache/pinot/pull/15891#issuecomment-2971794433
> @somandal When the UI/UX use of this is out, the API will be expected to used by the UI mainly, we can then deprecate the old usage. Is this fair? Also I did not find any case that deprecate the parameter of an API endpoint in pinot, may have to do more learning to do deprecation correctly, but I want to move on to other stuff to get UI/UX ready first. I am okay with doing this investigation later, but let's get that done before calling this project complete and updating documentation to say we'll be deprecating the other flags. Hope that's fine with you Reason I thought it is a good idea to do some investigation upfront on the best way to deprecate these parameters is because that might lead to a different implementation if there is no good path forward. It'll be harder to make that change later on if say we find out it's better to deprecate an API than parameters. That's fine though, in the worse case perhaps we just keep the flags around -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
