Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-17143 Scan improvement
  3. HBASE-15484

Correct the semantic of batch and partial

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Blocker
    • Resolution: Fixed
    • 1.4.0, 2.0.0
    • 1.4.0, 2.0.0
    • Client, Scanners
    • None
    • Reviewed
    • Hide
      Now setBatch doesn't mean setAllowPartialResult(true)
      If user setBatch(5) and rpc returns 3+5+5+5+3 cells, we should return 5+5+5+5+1 to user.
      Scan#setBatch is helpful in paging queries, if you just want to prevent OOM at client, use setAllowPartialResults(true) is better.
      We deprecated isPartial and use mayHaveMoreCellsInRow. If it returns false, current Result must be the last one of this row.
      Show
      Now setBatch doesn't mean setAllowPartialResult(true) If user setBatch(5) and rpc returns 3+5+5+5+3 cells, we should return 5+5+5+5+1 to user. Scan#setBatch is helpful in paging queries, if you just want to prevent OOM at client, use setAllowPartialResults(true) is better. We deprecated isPartial and use mayHaveMoreCellsInRow. If it returns false, current Result must be the last one of this row.

    Description

      Follow-up to HBASE-15325, as discussed, the meaning of setBatch and setAllowPartialResults should not be same. We should not regard setBatch as setAllowPartialResults.

      Now we deprecated isPartial and use mayHaveMoreCellsInRow. If it returns false, current Result must be the last one of this row.

      Attachments

        1. HBASE-15484.amend.patch
          6 kB
          Phil Yang
        2. HBASE-15484.branch-1.v01.patch
          33 kB
          Phil Yang
        3. HBASE-15484.v05.patch
          41 kB
          Phil Yang
        4. HBASE-15484.v06.patch
          41 kB
          Phil Yang
        5. HBASE-15484-v1.patch
          31 kB
          Phil Yang
        6. HBASE-15484-v2.patch
          32 kB
          Phil Yang
        7. HBASE-15484-v3.patch
          23 kB
          Phil Yang
        8. HBASE-15484-v4.patch
          31 kB
          Phil Yang

        Issue Links

          Activity

            People

              yangzhe1991 Phil Yang
              yangzhe1991 Phil Yang
              Votes:
              0 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: