Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13796 )

Change subject: KUDU-2722 (table level): Support new 'write_enabled' table 
extra config
......................................................................


Patch Set 8:

I will add some context to KUDU-2722 on why I thought partition level support 
would be useful/important. That said, it sounds like Xiaokai has a use case for 
table level support per our chats:

> Sometimes qps is over 10 thousand on partly nodes, which will cause some 
> problem such as    disk io. We should make sure that the table of important 
> task can be writed normally, so we will make the not much important table's 
> write is not enabled. When the heavy traffic is over, we will open again.

One thing I want to make sure we think about anytime we add a configuration, is 
if it is really needed. The more configuration Kudu has the harder it is to 
use, operate , and support.  I think adding this as experimental feature/config 
could make sense, but it sounds like adding better support for throttling, 
prioritization of tables/RPCs, and improved scalability would actually help 
solve the core issue more holistically.


--
To view, visit http://gerrit.cloudera.org:8080/13796
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd3768eda36d9574be9c41e7cd3dd81cc5ae8f3a
Gerrit-Change-Number: 13796
Gerrit-PatchSet: 8
Gerrit-Owner: XiaokaiWang <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: XiaokaiWang <[email protected]>
Gerrit-Reviewer: Yao Xu <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Comment-Date: Thu, 11 Jul 2019 14:40:48 +0000
Gerrit-HasComments: No

Reply via email to