Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-1198 Refactorings and Cleanups
  3. ZOOKEEPER-1251

call checkSession at begin of PrepRequestProcessor.pRequest

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Patch Available
    • Major
    • Resolution: Unresolved
    • None
    • None
    • None
    • None
    • Reviewed

    Description

      There are 6 locations that call checkSession. This can be reduced to one location and makes it also much clearer in which cases checkSession is called or not called.

      Note that in case that now the SessionMoved|Expired error is checked first before the check for a Marshalling error. However it shouldn't matter which error gets reported.

      Attachments

        1. ZOOKEEPER-1251_with_1208_and_1284.patch
          21 kB
          Thomas Koch
        2. update_to_1208_showingreadfailure.patch
          3 kB
          Patrick D. Hunt
        3. ZOOKEEPER-1251.patch
          4 kB
          Thomas Koch

        Activity

          People

            thkoch Thomas Koch
            thkoch Thomas Koch
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: