HBase
  1. HBase
  2. HBASE-8554

Scan should seek to startRow within a region

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Currently Scan.startRow() is only used for determining which Region to look up the row into but we do not seek within the region to the start row. Since its not uncommon to run with large sized regions these days 5G, this is suboptimal.

        Activity

        Hide
        Varun Sharma added a comment -

        I think this is tricky since we only want to seek for the first region and not the latter ones, but i still think this is doable by comparing with the current region's startKey and the Scan's startRow.

        Show
        Varun Sharma added a comment - I think this is tricky since we only want to seek for the first region and not the latter ones, but i still think this is doable by comparing with the current region's startKey and the Scan's startRow.
        Hide
        stack added a comment -

        Jean-Daniel Cryans Is this a regression? It can't have been this way always?

        Show
        stack added a comment - Jean-Daniel Cryans Is this a regression? It can't have been this way always?
        Hide
        Lars Hofhansl added a comment -

        I do not think I follow. What is the issue here?
        We do seek all the StoreScanners to the right place (see the StoreScanner constructor).

        Show
        Lars Hofhansl added a comment - I do not think I follow. What is the issue here? We do seek all the StoreScanners to the right place (see the StoreScanner constructor).
        Hide
        Varun Sharma added a comment -

        Agreed. Closing for now - I followed the response from Anoop and raised this issue... Thanks !

        Show
        Varun Sharma added a comment - Agreed. Closing for now - I followed the response from Anoop and raised this issue... Thanks !
        Hide
        Lars Hofhansl added a comment -

        That said, we should track down your performance issues... Continue that here or on the mailing list?

        Show
        Lars Hofhansl added a comment - That said, we should track down your performance issues... Continue that here or on the mailing list?
        Hide
        Anoop Sam John added a comment -

        Varun, I think I sent a half reply.. Sorry for the confusion. I never meant that startRow is just used in the client side only for determining the region.

        Show
        Anoop Sam John added a comment - Varun, I think I sent a half reply.. Sorry for the confusion. I never meant that startRow is just used in the client side only for determining the region.

          People

          • Assignee:
            Varun Sharma
            Reporter:
            Varun Sharma
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development