hadoop-zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick Hunt (JIRA)" <j...@apache.org>
Subject [jira] Created: (ZOOKEEPER-246) review error code definition in both source and docs
Date Fri, 05 Dec 2008 00:18:44 GMT
review error code definition in both source and docs
----------------------------------------------------

                 Key: ZOOKEEPER-246
                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-246
             Project: Zookeeper
          Issue Type: Improvement
          Components: c client, documentation, java client
    Affects Versions: 3.0.1, 3.0.0
            Reporter: Patrick Hunt
            Assignee: Patrick Hunt
            Priority: Minor
             Fix For: 3.1.0


A number of users have mentioned that our use of "calculated" values for error codes makes
it very difficult to track error reports from the logs.  In both c/java we have:

int ERROR = 100
int X_ERROR= ERROR + 1
int Y_ERROR= ERROR + 2
etc...

which makes it difficult to grep/search for "101" and find these references. A number of users
have mentioned that having non-calculated values (ie X_ERROR=101) would make it easier. 

I personally find calculated constants like this an anti-pattern.

We should convert these values to non-calculated at a minimum, perhaps even redefine them
as enums if possible.

Also documentation, esp in the c code (the names btw c and java don't match btw) is lacking.
We need to beef up docs in this area.


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


Mime
View raw message