You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by la...@apache.org on 2013/11/04 02:27:03 UTC
svn commit: r1538484 -
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
Author: larsh
Date: Mon Nov 4 01:27:02 2013
New Revision: 1538484
URL: http://svn.apache.org/r1538484
Log:
HBASE-8942 DFS errors during a read operation (get/scan), may cause write outliers (Amitanand Aiyer)
Modified:
hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java?rev=1538484&r1=1538483&r2=1538484&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java Mon Nov 4 01:27:02 2013
@@ -2200,20 +2200,15 @@ public class Store extends SchemaConfigu
* @throws IOException
*/
public KeyValueScanner getScanner(Scan scan,
- final NavigableSet<byte []> targetCols) throws IOException {
- lock.readLock().lock();
- try {
- KeyValueScanner scanner = null;
- if (getHRegion().getCoprocessorHost() != null) {
- scanner = getHRegion().getCoprocessorHost().preStoreScannerOpen(this, scan, targetCols);
- }
- if (scanner == null) {
- scanner = new StoreScanner(this, getScanInfo(), scan, targetCols);
- }
- return scanner;
- } finally {
- lock.readLock().unlock();
+ final NavigableSet<byte []> targetCols) throws IOException {
+ KeyValueScanner scanner = null;
+ if (getHRegion().getCoprocessorHost() != null) {
+ scanner = getHRegion().getCoprocessorHost().preStoreScannerOpen(this, scan, targetCols);
+ }
+ if (scanner == null) {
+ scanner = new StoreScanner(this, getScanInfo(), scan, targetCols);
}
+ return scanner;
}
@Override