#2315 Linked list for RI queue not efficient for many LDs

Performance
closed-fixed
5
2011-12-15
2011-12-13
No

Currently the Java CIM Client uses a linked list to maintain the relationship between the ServerIP/DestinationURL pair and corresponding reliable indication handler with its reordering queue. This is effective for a relatively small amount of listener destinations per WBEMListener, but would be a performance issue in Director environments where there could be up to 20,000 listener destinations.

Discussion

  • Dave Blaschke

    Dave Blaschke - 2011-12-13

    Adding a hash table as well and a property that will allow users to choose between the linked list and hash table:

    # The default value to use for the reliable indication handler's initial
    # hash table capacity. A value of 0 indicates use a linked list instead.
    # Linked lists are better suited for a small number of listener destinations
    # per WBEMListener while hash tables are better suited for a large number.
    #
    # Note: This property has no effect unless reliable indication support is
    # enabled. See the sblim.wbem.listenerEnableReliableIndications property.
    #
    # Type: Integer
    # Unit: Count
    # Recognition: On next creation of a WBEMListener
    # Range: 0 .. 25000
    # Default: 0

     
  • Dave Blaschke

    Dave Blaschke - 2011-12-13

    Patch sent for community review. During a 2 week period any
    exploiter may comment on the patch, request changes or turn it
    down completely (with good reason). For the time being the patch is part of the "Experimental" branch in CVS.

     
  • Dave Blaschke

    Dave Blaschke - 2011-12-13
    • status: open --> open-fixed
     
  • Dave Blaschke

    Dave Blaschke - 2011-12-13

    Review will be abbreviated due to release of 2.1.11 in two days. Current functionality is maintained if the new property, sblim.wbem.listenerReliableIndicationHashtableCapacity, is 0.

     
  • Dave Blaschke

    Dave Blaschke - 2011-12-13

    Patch against HEAD

     
  • Dave Blaschke

    Dave Blaschke - 2011-12-15

    The community review has completed and we received no substantial criticism. Therefore the patch has been approved and merged into the "HEAD" branch. The next release will pick it up.

     
  • Dave Blaschke

    Dave Blaschke - 2011-12-15
    • status: open-fixed --> pending-fixed
     
  • Dave Blaschke

    Dave Blaschke - 2011-12-15
    • status: pending-fixed --> closed-fixed
     
  • Dave Blaschke

    Dave Blaschke - 2011-12-15

    The patch was picked up by release 2.1.11 and will therefore be closed.

     

Log in to post a comment.