[
https://issues.apache.org/jira/browse/HBASE-17185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16382576#comment-16382576
]
Hadoop QA commented on HBASE-17185:
-----------------------------------
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m
0s{color} | {color:blue} Docker mode activated. {color} |
| {color:blue}0{color} | {color:blue} patch {color} | {color:blue} 0m
2s{color} | {color:blue} The patch file was not named according to hbase's
naming conventions. Please see
https://yetus.apache.org/documentation/0.7.0/precommit-patchnames for
instructions. {color} |
| {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 4s{color}
| {color:red} HBASE-17185 does not apply to master. Rebase required? Wrong
Branch? See https://yetus.apache.org/documentation/0.7.0/precommit-patchnames
for help. {color} |
\\
\\
|| Subsystem || Report/Notes ||
| JIRA Issue | HBASE-17185 |
| JIRA Patch URL |
https://issues.apache.org/jira/secure/attachment/12840991/HBASE-17185.v2.txt |
| Console output |
https://builds.apache.org/job/PreCommit-HBASE-Build/11760/console |
| Powered by | Apache Yetus 0.7.0 http://yetus.apache.org |
This message was automatically generated.
> Purge the seek of the next block reading HFileBlocks
> ----------------------------------------------------
>
> Key: HBASE-17185
> URL: https://issues.apache.org/jira/browse/HBASE-17185
> Project: HBase
> Issue Type: Improvement
> Components: HFile
> Affects Versions: 2.0.0
> Reporter: stack
> Assignee: stack
> Priority: Minor
> Labels: beginner
> Attachments: HBASE-17185.master.001.patch, HBASE-17185.patch,
> HBASE-17185.v2.txt
>
>
> When we read HFileBlocks, we read the asked-for block AND the next block's
> header which we add to a cache (see HBASE-17072). We do this extra read to
> get the next block's length purportedly. This seek of the next block's header
> complicates the HFileBlock construction (not to mind other consequences --
> again see HBASE-17072).
> Study done in HBASE-17072 shows that we normally do not need this extra read
> of the next block's header. In the usual case, the length of the block is
> gotten from the hfile index.
> A simplification of block reading can be done purging this extra header read.
> We can also save some space in cache.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
