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

chenxu commented on HBASE-23355:
--------------------------------

There also may be a scenario that we don't need cacheOnWrite and 
prefetchOnFlush, but prefetchOnRegionOpen is needed.
So how about consider both case?
if (cacheOnWrite || ! prefetchOnFlush)  {
  // bypass prefetch operations 
}

> Add a conf key to decide which operations can trigger prefetch
> --------------------------------------------------------------
>
>                 Key: HBASE-23355
>                 URL: https://issues.apache.org/jira/browse/HBASE-23355
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: chenxu
>            Assignee: chenxu
>            Priority: Major
>
> Copy the comment from anoop under 
> https://issues.apache.org/jira/browse/HBASE-23066?focusedCommentId=16984140&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16984140
> {quote}On a side note, (Not related to this issue) when we have cache on 
> write ON as well as prefetch also On, do we do the caching part for the 
> flushed files twice? When it is written, its already been added to cache. 
> Later as part of HFile reader open, the prefetch threads will again do a read 
> and add to cache!
> {quote}
> It's better to have a conf key to decide which operations(flush, compaction, 
> regionOpen, bulkload) can trigger the prefetch. In this way, if cacheonwrite 
> is enabled, we can exclude the flush and compaction operations when doing 
> prefetch.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to