HBase
  1. HBase
  2. HBASE-1954

Transactional scans do not see newest put.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.20.1
    • Fix Version/s: 0.20.2, 0.90.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      In a transaction, if I do a put, then a put, then a scan. I will not see the latest put.

      The fix is to set the timestamp at put time.

      1. 1954.patch
        2 kB
        Clint Morgan

        Activity

        Hide
        Clint Morgan added a comment -

        I saw a failure in TestTHLogRecovery in that Hudson output. Ran the test locally 2 times on both trunk and branch, and it passed all 4 times.

        (I don't think this patch would mess with how TTHLR is testing)

        Is something else failing? Is there a way I can reproduce?

        That TTHLR does need some work, and I'm planning on redoing in 0.21 after the core WAL stuff has settled down.

        Show
        Clint Morgan added a comment - I saw a failure in TestTHLogRecovery in that Hudson output. Ran the test locally 2 times on both trunk and branch, and it passed all 4 times. (I don't think this patch would mess with how TTHLR is testing) Is something else failing? Is there a way I can reproduce? That TTHLR does need some work, and I'm planning on redoing in 0.21 after the core WAL stuff has settled down.
        Hide
        Jean-Daniel Cryans added a comment -

        The build is unstable since this commit.

        Show
        Jean-Daniel Cryans added a comment - The build is unstable since this commit.
        Hide
        stack added a comment -

        Committed branch and trunk. Thanks for patch Clint.

        Show
        stack added a comment - Committed branch and trunk. Thanks for patch Clint.
        Hide
        Clint Morgan added a comment -

        This fixes the issue. Adds a test that fails without the patch, passes with.

        Show
        Clint Morgan added a comment - This fixes the issue. Adds a test that fails without the patch, passes with.

          People

          • Assignee:
            Clint Morgan
            Reporter:
            Clint Morgan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development