[GitHub] [incubator-doris] chaoyli commented on pull request #3883: [Enhancement] Add FORWARD_NO_SYNC for all SHOW PROC statements

[GitBox <gi...@apache.org>]

chaoyli commented on pull request #3883:
URL: https://github.com/apache/incubator-doris/pull/3883#issuecomment-644552857


   > 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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org