The SIO2 project
  1. The SIO2 project
  2. SIO-1682

Worker RPC implementation doesn't remove call from pending after timeout

        Details

        • Type: Bug Bug
        • Status: Closed Closed
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: Current Version
        • Fix Version/s: None
        • Component/s: Evaluation Engine / Workers
        • Labels:
          None

          Description

          sio.protocol.rpc.WorkerRpc._timeout will errback the Deferred, but the entry in self.pendingCalls is not deleted. If a reply actually comes later, callback() on line 71 will raise AlreadyCalledError, which will cause the connection to be dropped.

          This ironically makes SIO-1681 slightly better, so that one has to be fixed first.

            Issue Links

            is blocked by

            Bug - A problem which impairs or prevents the functions of the product. SIO-1681 sioworkersd doesn't handle timeouts or network errors properly

            • Minor - Minor loss of function, or other problem where easy workaround is present.
            • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.

              Activity

              Ascending order - Click to sort in descending order
              Bartosz Stebel made changes -
              Field Original Value New Value
              Link This issue is blocked by SIO-1681 [ SIO-1681 ]
              Hide
              Permalink
              Gerrit Gerrit added a comment -
              Change I8bbbd5e61a3b8c242cc30249b115089c0f1a71b9, patchset 1
              https://gerrit.sio2project.mimuw.edu.pl/2468

              SIO-1682 Worker RPC implementation doesn't remove call from pending after timeout

              Change-Id: I8bbbd5e61a3b8c242cc30249b115089c0f1a71b9
              Show
              Gerrit Gerrit added a comment - Change I8bbbd5e61a3b8c242cc30249b115089c0f1a71b9, patchset 1 https://gerrit.sio2project.mimuw.edu.pl/2468 SIO-1682 Worker RPC implementation doesn't remove call from pending after timeout Change-Id: I8bbbd5e61a3b8c242cc30249b115089c0f1a71b9
              Bartosz Stebel made changes -
              Status New [ 10000 ] Resolved [ 5 ]
              Assignee Bartosz Stebel [ implr ] Szymon Acedański [ accek ]
              Resolution Fixed [ 1 ]
              Szymon Acedański made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Transition Time In Source Status Execution Times Last Executer Last Execution Date
              New New Resolved Resolved
              		35d 11h 16m 1 Bartosz Stebel 2015-11-12 11:05
              Resolved Resolved Closed Closed
              		1230d 5h 34m 1 Szymon Acedański 2019-03-26 16:40

                People

                • Assignee:
                  Szymon Acedański
                  Reporter:
                  Bartosz Stebel
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: