[ 
https://issues.apache.org/jira/browse/HBASE-13456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14495911#comment-14495911
 ] 

Samir Ahmic commented on HBASE-13456:
-------------------------------------

Ted, Jerry thanks for review, 
bq. The full region name for the meta region is 'hbase:meta,,1', correct?
Agree, First meta region have old name format 
<tablename><startkey><regionIdTimestamp>, i think 
HRegionInfo.getRegionNameAsString() tricked me since it returns 
<tablename><startkey><regionIdTimestamp>.<encodedRegionName> and we use it in 
0.98 version to display region name on UI. I see that on master branch we use 
HRegionInfo.getRegionNameAsStringForDisplay()  for UI which displays correct 
name "hbase:meta,,1". So this is not issue for master branch. 

bg.  For the first problem you reported, should you fix the 
HFile.getStoreFiles() directly?
Let me check if we use HFile.getStoreFiles() somewhere else in code in other 
context and i if not i will fix HFile.getStoreFiles() directly.


> Improve HFilePrettyPrinter first hbase:meta region processing 
> --------------------------------------------------------------
>
>                 Key: HBASE-13456
>                 URL: https://issues.apache.org/jira/browse/HBASE-13456
>             Project: HBase
>          Issue Type: Improvement
>          Components: util
>    Affects Versions: 2.0.0, 0.98.12
>         Environment: x86_64 GNU/Linux
>            Reporter: Samir Ahmic
>            Assignee: Samir Ahmic
>            Priority: Minor
>         Attachments: HBASE-13456-0.98-v2.patch, HBASE-13456-0.98.patch, 
> HBASE-13456-master.patch
>
>
> HFilePrettyPrinter have couple of small issues when -r option is used with 
> hbase:meta region as argument:
> 1. Processing files in oldWals and recovered.edits directories
> {code}
> $ hbase org.apache.hadoop.hbase.io.hfile.HFilePrettyPrinter -v -r 
> hbase:meta,,1
> Number of region files found -> 3
> Found file[1] -> 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/info/e1d395908f8c43d2b22bb3a630728ebf
> Found file[2] -> 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/info/e30faa185198449b9cb2eeb5589ebd1b
> Found file[3] -> 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/oldWALs/hlog.1428309293399
> Scanning -> 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/info/e1d395908f8c43d2b22bb3a630728ebf
> Scanning -> 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/info/e30faa185198449b9cb2eeb5589ebd1b
> Scanning -> 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/oldWALs/hlog.1428309293399
> 2015-04-13 14:08:21,461 ERROR [main] hfile.HFilePrettyPrinter: Error reading 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/oldWALs/hlog.1428309293399
> org.apache.hadoop.hbase.io.hfile.CorruptHFileException: Problem reading HFile 
> Trailer from file 
> hdfs://P3cluster/hbase/data/hbase/meta/1588230740/oldWALs/hlog.1428309293399
> {code}
> 2. If full region name is passed as argument we have wrong encoded region 
> name returned:
> {code}
> hbase org.apache.hadoop.hbase.io.hfile.HFilePrettyPrinter -v -r 
> hbase:meta,,1.1588230740
> region dir -> hdfs://P3cluster/hbase/data/hbase/meta/104563571
> 2015-04-13 14:21:43,012 INFO  [main] Configuration.deprecation: 
> hadoop.native.lib is deprecated. Instead, use io.native.lib.available
> 2015-04-13 14:21:43,711 INFO  [main] util.ChecksumType: Checksum using 
> org.apache.hadoop.util.PureJavaCrc32
> 2015-04-13 14:21:43,717 INFO  [main] util.ChecksumType: Checksum can use 
> org.apache.hadoop.util.PureJavaCrc32C
> 2015-04-13 14:21:43,899 ERROR [main] hfile.HFilePrettyPrinter: Error parsing 
> command-line options
> java.io.FileNotFoundException: File 
> hdfs://P3cluster/hbase/data/hbase/meta/104563571 does not exist.
> {code}
> I have modified HFilePrettyPrinter.java to address this issues. Patch is 
> coming soon.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to