zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hanm <...@git.apache.org>
Subject [GitHub] zookeeper issue #279: ZOOKEEPER-2804:Node creation fails with NPE if ACLs ar...
Date Fri, 18 Aug 2017 04:18:14 GMT
Github user hanm commented on the issue:

    https://github.com/apache/zookeeper/pull/279
  
    Let's wrap this up before it becoming more stale. I believe the only remaining work item
is the last review comment @arshadmohammad made:
    >> As this Validation we are doing multiple places it would be better if this piece
of code is extracted to method. 
    @jainbhupendra24 Do you mind update this pull request and do what Arshad suggested?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message