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