Mailing-List: contact dev-help@airavata.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@airavata.apache.org
Received-SPF: pass (nike.apache.org: local policy)
Message-ID: <53591037.9020208@iu.edu>
Date: Thu, 24 Apr 2014 09:23:03 -0400
From: Marlon Pierce <marpierc@iu.edu>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6;
 rv:24.0) Gecko/20100101 Thunderbird/24.4.0
MIME-Version: 1.0
To: dev@airavata.apache.org
Subject: Re: Conveying Errors through the Thrift API
References: 
 <CANotcz5ruQpsSbuzU9CnLyQwnvDgmjL_40-7Je9+vw-bqTruow@mail.gmail.com>
In-Reply-To: 
 <CANotcz5ruQpsSbuzU9CnLyQwnvDgmjL_40-7Je9+vw-bqTruow@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

Hi Saminda, can you explain this in more detail:  "We felt that while
option 1 is ideal for the requirement it is also
unmanageable for both client and server if more and more exception types
needs to be introduced over time. "

What kinds of errors are we talking about?  What actions should the
client take?


Marlon

On 4/24/14 3:26 AM, Saminda Wijeratne wrote:
> During an offline discussion related to the JIRA[1], it was made clear that
> in an event of an exception at server side due to improper/invalid client
> requests, the client should get back a proper error message which can be
> programmed against. Two options were discussed,
>
>    1. Define exceptions in the thrift data model for each and every error
>    that can encounter and use them in the Thrift API
>    2. Define a generic exception in the thrift data model which will have
>    tagged data identifying the exception
>
> We felt that while option 1 is ideal for the requirement it is also
> unmanageable for both client and server if more and more exception types
> needs to be introduced over time. Option 2 (while not a conventional way of
> handling error) can circumvent this inconvenience by introducing a thrift
> function to retrieve exact static error details (which can be updated at
> the server side without needing to change the thrift model) based on the
> tagged data.
>
> wdyt?
>
> 1. https://issues.apache.org/jira/browse/AIRAVATA-1142
>