accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-2410) TServerUtils no longer needs to catch NPE as a thrift bug workaround
Date Thu, 12 Jun 2014 23:19:02 GMT

    [ https://issues.apache.org/jira/browse/ACCUMULO-2410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14029987#comment-14029987
] 

ASF subversion and git services commented on ACCUMULO-2410:
-----------------------------------------------------------

Commit c5aac49ed299b7c6eee534333cf080b638bcdecd in accumulo's branch refs/heads/1.5.2-SNAPSHOT
from [~al.krinker]
[ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c5aac49 ]

ACCUMULO-2410 TServerUtils no longer needs to catch NPE as a thrift bug workaround

Signed-off-by: John Vines <vines@apache.org>


> TServerUtils no longer needs to catch NPE as a thrift bug workaround
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-2410
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2410
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.5.1, 1.6.0
>            Reporter: Mike Drob
>            Assignee: Al Krinker
>            Priority: Minor
>              Labels: newbie
>             Fix For: 1.5.2, 1.6.1, 1.7.0
>
>         Attachments: ACCUMULO-2410.patch.txt
>
>
> Since we are using thrift 0.9.0 starting with Accumulo 1.6, we should be able to clean
up the following code in {{TServerUtils}}.
> {code:java}
>         try {
>           return other.process(in, out);
>         } catch (NullPointerException ex) {
>           // THRIFT-1447 - remove with thrift 0.9
>           return true;
>         }
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message