shounakmk219 opened a new pull request, #12417: URL: https://github.com/apache/pinot/pull/12417
# Description This PR aims to allow a way of passing the database context along with the API requests without disrupting the existing APIs. The database context will be passed as a `database` http header along with the API requests. It is part of the effort to support database concept in Pinot #12333 To avoid writing the logic for the translation `databaseName` + `tableName` -> `databaseName.tableName` for each API this PR also introduces a request filter for controller and broker endpoints which pre computes the fully qualified table name for the APIs, only condition being that the tableName must be a query param. APIs with tableName input as path param and request body will still need to do the translation on their own. In order to reduce the explicit handling of table translation on each API, this PR also introduces a set of V2 endpoints for APIs which take tableName input as path param and request body. More details [here](https://docs.google.com/document/d/1l1dco4lRhB28y-yKMVKlOMZ0sdFKObjJfhAjG6cmktU/edit#heading=h.plsbi2ybsb2b) Any new API that get introduced and needs table name as input should follow this convention for table name input. # Release Notes - Allow passing database name context through `database` http header - V2 endpoints introduced which take table name input as query param -- 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
