[ 
https://issues.apache.org/jira/browse/ACCUMULO-3842?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14556765#comment-14556765
 ] 

Keith Turner commented on ACCUMULO-3842:
----------------------------------------

bq. ZooKeeper doesn't keep us from accomplishing this.  We would need to write 
code to actually get the strong consensus for ourselves.

Isn't this statement is true for HDFS also?   Wether using zookeeper or HDFS, 
config needs to be cached on tservers for efficiency.  For most operations 
config is not changing, do not want to make a synchronous read to ZK of HDFS 
before servicing each RPC.  I think we need something at the API level to 
address this, regardless of implementation.

bq.  but I think we're at the point where this is a problem we need to start 
thinking about because it's been a repeated problem for ourselves just in 
writing reasonable tests for Accumulo for ~2years now.

Yes, this is a big problem, lets fix it.   I think ZK provides primitives that 
make adding an Accumulo API to wait for tservers to synch config to a certain 
point fairly easy to implement.  I think a good place to start is thinking what 
the API solutions for this would look like.  Some possible API changes :

 # Add something like  {{waitForConfigSynch(long timeout, TimeUnit tu)}} to the 
API.  If it times out could throw an exception w/ info about the tservers that 
did not update.
 # Change behavior of  current methods that change config to wait for all 
tservers to sync (I don't like doing this, would rather leave the current 
behavoir).
 # Add a new method that allows setting a batch of key/value configs.  This 
method would also wait for changes to sync across tservers.   Something like 
{{updateConfiguration(Map<String, String> propsToSet, Set<String> propsToUnset, 
long timeout, TimeUnit tu)}}.  Of course pass in class instead of 4 params.

I kinda like 3. What other API changes could solve this?

> [UMBRELLA] Remove non-transient data from ZooKeeper
> ---------------------------------------------------
>
>                 Key: ACCUMULO-3842
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3842
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client, tserver
>            Reporter: Josh Elser
>             Fix For: 1.8.0
>
>
> Wanted to start brainstorming about this.
> We store a lot of persistent data in ZooKeeper that would better stored in 
> something backed by HDFS. ZooKeeper can be a very convenient place to store 
> persisted data so that it's available to all nodes, but it comes at a price 
> and often must be asynchronously accessed to achieve good performance.
> * Table/Namespace configuration
> * Users/Authorizations
> * Problem reports (maybe?)
> * System configuration overrides (maybe?)
> Some benefits we'd see from this:
> * Loss of ZooKeeper doesn't lose table configuration and users.
> * Greatly reduce zookeeper watchers (assume 
> watchers=50*num_tables*num_tservers)
> * Consistent updates of table constraints and all other table properties
> The last note is the most important one IMO. The number of test issues alone 
> that we've had with constraints not being seen on all servers are bound to 
> affect users.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to