HBase
  1. HBase
  2. HBASE-3813

Change RPC callQueue size from "handlerCount * MAX_QUEUE_SIZE_PER_HANDLER;"

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.92.0
    • Fix Version/s: 0.92.0
    • Component/s: None
    • Labels:
      None

      Description

      Yesterday debugging w/ Jack we noticed that with few handlers on a big box, he was seeing stats like this:

      2011-04-21 11:54:49,451 DEBUG org.apache.hadoop.ipc.HBaseServer: Server connection from X.X.X.X:60931; # active connections: 11; # queued calls: 2500
      

      We had 2500 items in the rpc queue waiting to be processed.

      Turns out he had too few handlers for number of clients (but also, it seems like he figured hw issues in that his RAM bus was running at 1/4 the rate that it should have been running at).

      Chatting w/ J-D this morning, he asked if the queues hold 'data'. The queues hold 'Calls'. Calls are the client request. They contain data.

      Jack had 2500 items queued. If each item to insert was 1MB, thats 25k * 1MB of memory that is outside of our generally accounting.

      Currently the queue size is handlers * MAX_QUEUE_SIZE_PER_HANDLER where MAX_QUEUE_SIZE_PER_HANDLER is hardcoded to be 100.

      If the queue is full we block (LinkedBlockingQueue).

      Going to change the queue size from 100 to 10 by default – but also will make it configurable and will doc. this as possible cause of OOME. Will try it on production here before committing patch.

        Activity

        stack created issue -
        stack made changes -
        Field Original Value New Value
        Attachment 3813.txt [ 12478247 ]
        stack made changes -
        Affects Version/s 0.92.0 [ 12314223 ]
        Affects Version/s 0.90.3 [ 12316313 ]
        stack made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.92.0 [ 12314223 ]
        Resolution Fixed [ 1 ]
        Jean-Daniel Cryans made changes -
        Assignee stack [ stack ]

          People

          • Assignee:
            stack
            Reporter:
            stack
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development