You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/06/13 01:19:02 UTC
[jira] [Commented] (ACCUMULO-2410) TServerUtils no longer needs to
catch NPE as a thrift bug workaround
[ 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 <vi...@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)