River
  1. River
  2. RIVER-397

Mux.start() can block indefinitely

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: jtsk_2.1
    • Fix Version/s: River_2.2.0
    • Component/s: net_jini_jeri
    • Labels:
      None

      Description

      Email thread starting here: http://mail-archives.apache.org/mod_mbox/river-dev/201104.mbox/%3C77F1E32F67C8D5479858C0C7E93EB465068B06CF@WAL-MAIL.global.avidww.com%3E

      In short, I encountered a case where a JERI client got permanently stuck in Mux.start(). Attached is a patch to add a timeout to that method.

      1. RIVER-397.patch
        3 kB
        Chris Dolan
      2. MuxStartTimeout.java
        2 kB
        Chris Dolan

        Activity

        Hide
        Tom Hobbs added a comment -

        Moving to River 2.2.0

        Show
        Tom Hobbs added a comment - Moving to River 2.2.0
        Hide
        Hudson added a comment -

        Integrated in River-trunk #492 (See https://builds.apache.org/job/River-trunk/492/)
        River-397 Test case.

        Show
        Hudson added a comment - Integrated in River-trunk #492 (See https://builds.apache.org/job/River-trunk/492/ ) River-397 Test case.
        Hide
        Peter Firmstone added a comment -

        Test case and patch supplied committed.

        Show
        Peter Firmstone added a comment - Test case and patch supplied committed.
        Hide
        Chris Dolan added a comment -

        Test case that fails with current River and passes with the proposed timeout solution applied.

        Show
        Chris Dolan added a comment - Test case that fails with current River and passes with the proposed timeout solution applied.
        Hide
        Chris Dolan added a comment -

        proposed solution with configurable timeout for Mux.start()

        Show
        Chris Dolan added a comment - proposed solution with configurable timeout for Mux.start()

          People

          • Assignee:
            Unassigned
            Reporter:
            Chris Dolan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development