HBase
  1. HBase
  2. HBASE-3572

memstore lab can leave half inited data structs (bad!)

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.90.1, 0.92.0
    • Fix Version/s: 0.90.2, 0.92.0
    • Component/s: None
    • Labels:
      None

      Description

      in Chunk.init() if new byte[] fails it leaves the Chunk in its uninitialized state, other threads will assume someone else will init it and get stuck in an infinite loop.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        2m 11s 1 ryan rawson 26/Feb/11 07:10
        Patch Available Patch Available Resolved Resolved
        2d 12h 6m 1 ryan rawson 28/Feb/11 19:17
        Hide
        Hudson added a comment -

        Integrated in HBase-TRUNK #1767 (See https://hudson.apache.org/hudson/job/HBase-TRUNK/1767/)

        Show
        Hudson added a comment - Integrated in HBase-TRUNK #1767 (See https://hudson.apache.org/hudson/job/HBase-TRUNK/1767/ )
        ryan rawson made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Todd Lipcon added a comment -

        +1 lgtm, nice catch

        Show
        Todd Lipcon added a comment - +1 lgtm, nice catch
        Hide
        ryan rawson added a comment -

        unfortunately you end up with a handler in an infinite loop, and that
        keeps the server from shutting down.

        Show
        ryan rawson added a comment - unfortunately you end up with a handler in an infinite loop, and that keeps the server from shutting down.
        Hide
        stack added a comment -

        Patch looks good to me, apply it I'd say, but ain't we hosed if we OOME in first place?

        Show
        stack added a comment - Patch looks good to me, apply it I'd say, but ain't we hosed if we OOME in first place?
        ryan rawson made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Affects Version/s 0.90.1 [ 12315548 ]
        Affects Version/s 0.92.0 [ 12314223 ]
        Fix Version/s 0.90.2 [ 12316152 ]
        Fix Version/s 0.92.0 [ 12314223 ]
        ryan rawson made changes -
        Attachment memstorelab-oom.txt [ 12472018 ]
        ryan rawson made changes -
        Field Original Value New Value
        Assignee ryan rawson [ ryanobjc ]
        ryan rawson created issue -

          People

          • Assignee:
            ryan rawson
            Reporter:
            ryan rawson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development