Accumulo
  1. Accumulo
  2. ACCUMULO-2410

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

    Details

      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

          Activity

          Hide
          Josh Elser added a comment -

          Thrift-0.9.0 is used by both 1.5 and 1.6. 1.4 is the only version still on Thrift 0.6.x

          Show
          Josh Elser added a comment - Thrift-0.9.0 is used by both 1.5 and 1.6. 1.4 is the only version still on Thrift 0.6.x
          Hide
          ASF subversion and git services added a comment -

          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>

          Show
          ASF subversion and git services added a comment - 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>
          Hide
          ASF subversion and git services added a comment -

          Commit c5aac49ed299b7c6eee534333cf080b638bcdecd in accumulo's branch refs/heads/1.6.1-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>

          Show
          ASF subversion and git services added a comment - Commit c5aac49ed299b7c6eee534333cf080b638bcdecd in accumulo's branch refs/heads/1.6.1-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>
          Hide
          ASF subversion and git services added a comment -

          Commit c5aac49ed299b7c6eee534333cf080b638bcdecd in accumulo's branch refs/heads/master 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>

          Show
          ASF subversion and git services added a comment - Commit c5aac49ed299b7c6eee534333cf080b638bcdecd in accumulo's branch refs/heads/master 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>
          Hide
          John Vines added a comment -

          Patch applied, thanks Al!

          Show
          John Vines added a comment - Patch applied, thanks Al!

            People

            • Assignee:
              Al Krinker
              Reporter:
              Mike Drob
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development