XiaokaiWang 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.

Ok, got. Let's ignore the feature.


--
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 15:53:34 +0000
Gerrit-HasComments: No

Reply via email to