morningman commented on pull request #3883: URL: https://github.com/apache/incubator-doris/pull/3883#issuecomment-644543498
Hi @chaoyli . Actually, not all `ShowProc` stmt should be forward to master. Some of them, yes, like: `show proc "/cluster_balance"`, because the non-master FE has nothing to do the the cluster balance logic. But some of them don't, like: `show proc "/dbs/10003/10017/partitions/10016/10018";`. This is the stmt to show the tablet info saving in the current FE's memory. These kind of `show proc` stmt should not be forward to master FE because sometimes we need to check the meta of current FE, to see if there is any inconsistency with the Master FE. That is why I add the variable `forward_to_master` to control whether to forward these stmts. ---------------------------------------------------------------- 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...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
