HBase
  1. HBase
  2. HBASE-3476

HFile -m option need not scan key values

    Details

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

      Description

      bin/hbase org.apache.hadoop.io.hfile.HFile -m -f <filename> doesn't have to scan the KVs in the file

        Activity

        Hide
        Hudson added a comment -

        Integrated in HBase-TRUNK #1721 (See https://hudson.apache.org/hudson/job/HBase-TRUNK/1721/)
        HBASE-3476 HFile -m option need not scan key values (Prakash Khemani via Lars George)

        Show
        Hudson added a comment - Integrated in HBase-TRUNK #1721 (See https://hudson.apache.org/hudson/job/HBase-TRUNK/1721/ ) HBASE-3476 HFile -m option need not scan key values (Prakash Khemani via Lars George)
        Hide
        Lars George added a comment -

        I just committed this. Thanks Prakash.

        Show
        Lars George added a comment - I just committed this. Thanks Prakash.
        Hide
        Lars George added a comment -

        Somehow the RB integration is borked. Patch looks good, committing since this is trivial.

        Show
        Lars George added a comment - Somehow the RB integration is borked. Patch looks good, committing since this is trivial.
        Hide
        Prakash Khemani added a comment -

        I had put up this diff https://review.cloudera.org/r/1489/ . I am not sure why it didn’t get propagated to the JIRA.

        Please feel free to put your own patch and close this issue.

        Thanks.

        Show
        Prakash Khemani added a comment - I had put up this diff https://review.cloudera.org/r/1489/ . I am not sure why it didn’t get propagated to the JIRA. Please feel free to put your own patch and close this issue. Thanks.
        Hide
        Lars George added a comment -

        I agree, this is an oversight. We should add an extra check and skip the scan if not needed. Do you want to submit a patch or do you want me to fix it?

        Show
        Lars George added a comment - I agree, this is an oversight. We should add an extra check and skip the scan if not needed. Do you want to submit a patch or do you want me to fix it?

          People

          • Assignee:
            Prakash Khemani
            Reporter:
            Prakash Khemani
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development