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

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.5.1, 1.6.0
• Fix Version/s: 1.5.2, 1.6.1, 1.7.0
• Component/s: tserver
• Labels: Summit2014 newbie
• Assignee: Al Krinker
• Reporter: Mike Drob
• Created: 26/Feb/14 19:15
• Updated: 12/Jun/14 23:20
• Resolved: 12/Jun/14 23:20

Description

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.

        try {
          return other.process(in, out);
        } catch (NullPointerException ex) {
          // THRIFT-1447 - remove with thrift 0.9
          return true;
        }

Issue Links

• depends upon: ACCUMULO-822 upgrade trunk to thrift 0.9
• depends upon: THRIFT-1447 NullpointerException in ProcessFunction.class :in "oneway" method