You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Michael Han (JIRA)" <ji...@apache.org> on 2016/11/10 17:19:58 UTC

[jira] [Created] (ZOOKEEPER-2630) Use interface type instead of implementation type when appropriate.

Michael Han created ZOOKEEPER-2630:
--------------------------------------

             Summary: Use interface type instead of implementation type when appropriate.
                 Key: ZOOKEEPER-2630
                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2630
             Project: ZooKeeper
          Issue Type: Improvement
            Reporter: Michael Han
            Priority: Trivial


There are a couple of places in code base where we declare a field / variable as implementation type (i.e. HashMap, HashSet) instead of interface type (i.e. Map, Set), while in other places we do the opposite by declaring as interface type. A quick check indicates that most if not all of these places could be updated so we have a consistent style over the code base (prefer using interface type), which is also a good coding style to stick per best practice.

See more info on https://github.com/apache/zookeeper/pull/102



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