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

Ashu Pachauri commented on HBASE-15890:
---------------------------------------

Yes, it is. But, test for setting "cacheBlocks" for scans already exists in 
org.apache.hadoop.hbase.regionserver.TestBlocksRead#testBlocksStoredWhenCachingDisabled.
 I don't want to add another redundant end-to-end test. I can add a test for 
whether the property is successfully passed on from TScan to Scan, not sure who 
much value that adds given how straight forward the change is.

> Allow thrift to set/unset "cacheBlocks" for Scans
> -------------------------------------------------
>
>                 Key: HBASE-15890
>                 URL: https://issues.apache.org/jira/browse/HBASE-15890
>             Project: HBase
>          Issue Type: Improvement
>          Components: Thrift
>            Reporter: Ashu Pachauri
>            Assignee: Ashu Pachauri
>             Fix For: 2.0.0, 1.3.0
>
>         Attachments: HBASE-15890-V0.patch
>
>
> Long running scans going through thrift cache everything to the block cache. 
> We need the ability to disable caching for scans going through thrift.



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

Reply via email to