Thrift
  1. Thrift
  2. THRIFT-957

THsHaServer: Change access modifier of the invoker field.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6, 0.7
    • Fix Version/s: 0.6
    • Component/s: Java - Library
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      I would like to subclass the THsHaServer so that I can override the protected requestInvoke method. I need to implement my own Runnable using the invoker.

      This is hard to do cleanly without the ability to access the invoker field.

      1. THRIFT-957.patch
        1 kB
        Benjamin Coverston

        Activity

        Bryan Duxbury made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Assignee Benjamin Coverston [ bcoverston ]
        Fix Version/s 0.7 [ 12315360 ]
        Resolution Fixed [ 1 ]
        Benjamin Coverston made changes -
        Attachment thrift-trunk-957.txt [ 12457220 ]
        Benjamin Coverston made changes -
        Attachment THRIFT-957.patch [ 12457579 ]
        Benjamin Coverston made changes -
        Fix Version/s 0.5 [ 12315235 ]
        Affects Version/s 0.4 [ 12314960 ]
        Affects Version/s 0.5 [ 12315235 ]
        Description I would like to subclass the THsHaServer so that I can override the protected requestInvoke method. I need to implement my own Runnable with the declared ExecutorService. This is hard to do cleanly without the ability to access the invoker field.
        I would like to subclass the THsHaServer so that I can override the protected requestInvoke method. I need to implement my own Runnable using the invoker.

         This is hard to do cleanly without the ability to access the invoker field.

        Benjamin Coverston made changes -
        Field Original Value New Value
        Attachment thrift-trunk-957.txt [ 12457220 ]
        Benjamin Coverston created issue -

          People

          • Assignee:
            Benjamin Coverston
            Reporter:
            Benjamin Coverston
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2h
              2h
              Remaining:
              Remaining Estimate - 2h
              2h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development