hadoop-hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ted Yu <yuzhih...@gmail.com>
Subject Re: question on TProtocol exception handling
Date Fri, 09 Apr 2010 21:40:04 GMT
Here is the patch.


On Fri, Apr 9, 2010 at 2:33 PM, John Sichi <jsichi@facebook.com> wrote:

> Hi Ted,
>
> I'm not familiar with this code--maybe someone else from hive-dev knows the
> answer, but it looks like Thrift-generated code (I can't access the patch at
> the moment because JIRA seems to be inaccessible).
>
> JVS
>
> On Apr 9, 2010, at 11:36 AM, Ted Yu wrote:
>
> John:
>
> I see the following code in hive- 417-2009-07-18.patch<https://issues.apache.org/jira/secure/attachment/12413928/hive-417%EF%BC%8D2009-07-18.patch>
> :
> +    private class create_index implements ProcessFunction {
> +      public void process(int seqid, TProtocol iprot, TProtocol oprot)
> throws TException
> +      {
> +        create_index_args args = new create_index_args();
> +        args.read(iprot);
> +        iprot.readMessageEnd();
> +        create_index_result result = new create_index_result();
> +        try {
> +          iface_.create_index(args.
> index);
> +        } catch (AlreadyExistsException o1) {
> +          result.o1 = o1;
> +        } catch (InvalidObjectException o2) {
> +          result.o2 = o2;
>
> What's the rational behind using multiple fields to send back exception
> information ?
>
> Thanks
>
>
>

Mime
  • Unnamed multipart/mixed (inline, None, 0 bytes)
View raw message