malbx commented on pull request #6765: URL: https://github.com/apache/incubator-pinot/pull/6765#issuecomment-820919469
> `SERVER_TABLE_MISSING_ERROR` means the table exists, but broker routes the query to a server that does not have the table loaded. It is a system internal error. > We need to add a new error type for non-existing table, and use `TableCache` to check if the table config exist The new error type for non-existing table has been added. Here is the current code that appears to determine if the table exist. https://github.com/apache/incubator-pinot/blob/aaa28fa0446507704dfa3e4b58335a41f5c1b430/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java#L294-L300 Are you suggesting that this code should use `TableCache` to check if the table exist instead of the current approach? -- 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. 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
