ankitsultana commented on PR #16677: URL: https://github.com/apache/pinot/pull/16677#issuecomment-3237931266
> > on high level, what's the difference between preview endpoint and the existing validation endpoint? > > The create endpoint does some tuning of the config (see [tuneConfig](https://github.com/apache/pinot/blob/274fe480617bf1404770d930300fcc72d9a7e63e/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/TableConfigsRestletResource.java#L481)). That is the config that is actually used. Validate does not do that. There's also other validations that happen as a part of create like tenant existence checks etc that are not a part of the validateConfig() call Another option is to extend the responsibility of the validate endpoint to return a preview of the config in a new JSON field. That would prevent introduction of another endpoint which is quite similar to the validate endpoint. -- 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]
