You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by mb...@apache.org on 2014/02/28 10:16:53 UTC

svn commit: r1572891 - /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/io/FileLink.java

Author: mbertozzi
Date: Fri Feb 28 09:16:52 2014
New Revision: 1572891

URL: http://svn.apache.org/r1572891
Log:
HBASE-10631 Avoid extra seek on FileLink open

Modified:
    hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/io/FileLink.java

Modified: hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/io/FileLink.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/io/FileLink.java?rev=1572891&r1=1572890&r2=1572891&view=diff
==============================================================================
--- hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/io/FileLink.java (original)
+++ hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/io/FileLink.java Fri Feb 28 09:16:52 2014
@@ -287,7 +287,7 @@ public class FileLink {
         if (path.equals(currentPath)) continue;
         try {
           in = fs.open(path, bufferSize);
-          in.seek(pos);
+          if (pos != 0) in.seek(pos);
           assert(in.getPos() == pos) : "Link unable to seek to the right position=" + pos;
           if (LOG.isTraceEnabled()) {
             if (currentPath != null) {