gortiz commented on PR #18317: URL: https://github.com/apache/pinot/pull/18317#issuecomment-4420736333
> The issue is that when we're manipulating a particular index in APIs/DP, we are inferring the config eventually as IndexConfig, and not plain jsonNode, which when inferred, gets populated with fat config, and then when applied with any modification, results in full fat config overriding the initial slim configuration. That is an issue of the API, not internal to Pinot. A simpler and less invasive solution would be to simplify the received new config when it is received. For example, we can add an extra option to the POST method to indicate whether the index configs should be simplified or not. Or even better, you app, which is the one bloating the config, could check whether the value being set is the same as the default and in that case simplify the JSON that will be sent. -- 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]
