chaoyli commented on pull request #3883: URL: https://github.com/apache/incubator-doris/pull/3883#issuecomment-644554315
> 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. I thinks the variable forward_to_master is useless, even make application complicated. User deploy the Doris will one Master and two Follower. The master will change when new vote. And I don't know whether to set the variable or not. The user must remember every show statement which it's semantics. ---------------------------------------------------------------- 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
