svn commit: r1663508 - in /pig/trunk: CHANGES.txt src/org/apache/pig/backend/hadoop/datastorage/HPath.java

[daijy]

Author: daijy
Date: Tue Mar  3 01:21:35 2015
New Revision: 1663508

URL: http://svn.apache.org/r1663508
Log:
PIG-4442: Eliminate redundant RPC call to get file information in HPath

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/backend/hadoop/datastorage/HPath.java

Modified: pig/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1663508&r1=1663507&r2=1663508&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Tue Mar  3 01:21:35 2015
@@ -50,6 +50,8 @@ PIG-4333: Split BigData tests into multi
  
 BUG FIXES
 
+PIG-4442: Eliminate redundant RPC call to get file information in HPath 
(cnauroth via daijy)
+
 PIG-4440: Some code samples in documentation use Unicode left/right single 
quotes, which cause a
  parse failure (cnauroth via daijy)
 

Modified: pig/trunk/src/org/apache/pig/backend/hadoop/datastorage/HPath.java
URL: 
http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/backend/hadoop/datastorage/HPath.java?rev=1663508&r1=1663507&r2=1663508&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/backend/hadoop/datastorage/HPath.java 
(original)
+++ pig/trunk/src/org/apache/pig/backend/hadoop/datastorage/HPath.java Tue Mar  
3 01:21:35 2015
@@ -124,10 +124,11 @@ public abstract class HPath implements E
 
     public Properties getConfiguration() throws IOException {
         HConfiguration props = new HConfiguration();
+        FileStatus fileStatus = fs.getHFS().getFileStatus(path);
 
-        long blockSize = fs.getHFS().getFileStatus(path).getBlockSize();
+        long blockSize = fileStatus.getBlockSize();
 
-        short replication = fs.getHFS().getFileStatus(path).getReplication();
+        short replication = fileStatus.getReplication();
 
         props.setProperty(BLOCK_SIZE_KEY, 
(Long.valueOf(blockSize)).toString());
         props.setProperty(BLOCK_REPLICATION_KEY, 
(Short.valueOf(replication)).toString());