Mailing-List: contact pig-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: pig-dev@hadoop.apache.org
Message-ID: <27369497.186981284493053592.JavaMail.jira@thor>
Date: Tue, 14 Sep 2010 15:37:33 -0400 (EDT)
From: "Thejas M Nair (JIRA)" <jira@apache.org>
To: pig-dev@hadoop.apache.org
Subject: [jira] Created: (PIG-1611) use enums for error code
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

use enums for error code
------------------------

                 Key: PIG-1611
                 URL: https://issues.apache.org/jira/browse/PIG-1611
             Project: Pig
          Issue Type: Sub-task
            Reporter: Thejas M Nair
             Fix For: 0.9.0


Pig code is using integer constants for error code, and the value of the error code is reserved using http://wiki.apache.org/pig/PigErrorHandlingFunctionalSpecification .
This process is cumbersome and error prone.

It will be better to use enum values instead. The enum value can contain the error message and encapsulate the error code. 
For example -
{code}
Replace 
throw new SchemaMergeException("Error in merging schema", 2124, PigException.BUG); 
with
throw new SchemaMergeException(SCHEMA_MERGE_EX, PigException.BUG); 

{code}


Where SCHEMA_MERGE_EX belongs to a error codes enum. We can use the ordinal value of the enum and an offset to determine the error code. 
The error code will be passed through the constructor of the enum.
{code}
SCHEMA_MERGE_EX("Error in merging schema");
{code}

For documentation, the error code and error messages can be dumped using code that uses the enum error code class.


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