You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Datko (JIRA)" <ji...@apache.org> on 2010/08/11 16:59:18 UTC

[jira] Updated: (ZOOKEEPER-840) massive code duplication in zookeeper class

     [ https://issues.apache.org/jira/browse/ZOOKEEPER-840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Datko updated ZOOKEEPER-840:
------------------------------------

           Tags: code duplication
    Description: 
Each operation calls validatePath, handles the chroot, calls ClientCnxn and 
checks the return header for error. I'd like to address this with the 
operation classes:
Each operation should receive a prechecked Path object. Calling ClientCnxn and 
error checking is not (or only partly) the concern of the operation but of an 
"executor" like class.

> massive code duplication in zookeeper class
> -------------------------------------------
>
>                 Key: ZOOKEEPER-840
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-840
>             Project: Zookeeper
>          Issue Type: Sub-task
>            Reporter: Patrick Datko
>
> Each operation calls validatePath, handles the chroot, calls ClientCnxn and 
> checks the return header for error. I'd like to address this with the 
> operation classes:
> Each operation should receive a prechecked Path object. Calling ClientCnxn and 
> error checking is not (or only partly) the concern of the operation but of an 
> "executor" like class.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.