Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.5.0
    • Fix Version/s: 0.5.0
    • Component/s: bsp core
    • Labels:
      None

      Description

      After HAMA-521 is done, we can add a Heap-based queue which sorts the incoming messages by a given Comparator. So the message should be WritableComparable, note that this should be not a requirement in the generics of BSP since this is just a corner-case.

      1. HAMA-552.patch
        3 kB
        Thomas Jungblut

        Issue Links

          Activity

          Suraj Menon made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Hide
          Suraj Menon added a comment -

          Thanks for the contribution.

          Show
          Suraj Menon added a comment - Thanks for the contribution.
          Hide
          Hudson added a comment -

          Integrated in Hama-Nightly #550 (See https://builds.apache.org/job/Hama-Nightly/550/)
          HAMA-552: Add a sorted message queue (Revision 1339626)

          Result = SUCCESS
          tjungblut :
          Files :

          • /incubator/hama/trunk/CHANGES.txt
          • /incubator/hama/trunk/core/src/main/java/org/apache/hama/bsp/message/SortedMessageQueue.java
          Show
          Hudson added a comment - Integrated in Hama-Nightly #550 (See https://builds.apache.org/job/Hama-Nightly/550/ ) HAMA-552 : Add a sorted message queue (Revision 1339626) Result = SUCCESS tjungblut : Files : /incubator/hama/trunk/CHANGES.txt /incubator/hama/trunk/core/src/main/java/org/apache/hama/bsp/message/SortedMessageQueue.java
          Thomas Jungblut made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Thomas Jungblut made changes -
          Fix Version/s 0.5.0 [ 12316942 ]
          Fix Version/s 0.6.0 [ 12319740 ]
          Hide
          Thomas Jungblut added a comment -

          Put that into trunk now.

          Show
          Thomas Jungblut added a comment - Put that into trunk now.
          Thomas Jungblut made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Thomas Jungblut made changes -
          Attachment HAMA-552.patch [ 12525461 ]
          Hide
          Thomas Jungblut added a comment -

          Testcases are fine, rat also. I don't think this here needs a testcase, it is just a delegate to the java implementation.

          Show
          Thomas Jungblut added a comment - Testcases are fine, rat also. I don't think this here needs a testcase, it is just a delegate to the java implementation.
          Thomas Jungblut made changes -
          Assignee Thomas Jungblut [ thomas.jungblut ]
          Thomas Jungblut made changes -
          Link This issue is cloned as HAMA-559 [ HAMA-559 ]
          Thomas Jungblut made changes -
          Field Original Value New Value
          Link This issue is blocked by HAMA-521 [ HAMA-521 ]
          Thomas Jungblut created issue -

            People

            • Assignee:
              Thomas Jungblut
              Reporter:
              Thomas Jungblut
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development