tibrewalpratik17 commented on issue #12273: URL: https://github.com/apache/pinot/issues/12273#issuecomment-1905862904
> i think an allowed list would be desirable. but it is a bit hard to make it configurable/extensible without overriding it. do we have any idea on how one can extend this to support extra properties? should we allow cluster config modification (e.g. not as a static broker conf file but modifiable during runtime) We need this info at `RequestHandler` class when pushing tracked-headers to `RequestContext`. To make it configurable: - we will either have to recreate `RequestHandler` classes if this list is passed as a param. - store it as a static reference somewhere and create a controller API to update these properties for a set of component. The default value can be read from that component's config file. But after updating, we might need to push the updated list to some conf file again so that restarts don't affect the list? -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
