You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lens.apache.org by ra...@apache.org on 2015/10/09 06:17:30 UTC

[25/50] [abbrv] lens git commit: LENS-775 : NPE on closing HadoopFileFormatter

LENS-775 : NPE on closing HadoopFileFormatter


Project: http://git-wip-us.apache.org/repos/asf/lens/repo
Commit: http://git-wip-us.apache.org/repos/asf/lens/commit/f4c6c11e
Tree: http://git-wip-us.apache.org/repos/asf/lens/tree/f4c6c11e
Diff: http://git-wip-us.apache.org/repos/asf/lens/diff/f4c6c11e

Branch: refs/heads/current-release-line
Commit: f4c6c11e9637107a63493b98e3bb7d50966e52c1
Parents: 3862c38
Author: Deepak Barr <de...@gmail.com>
Authored: Thu Sep 17 19:12:17 2015 +0530
Committer: Rajat Khandelwal <ra...@gmail.com>
Committed: Thu Sep 17 19:12:17 2015 +0530

----------------------------------------------------------------------
 .../main/java/org/apache/lens/lib/query/HadoopFileFormatter.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lens/blob/f4c6c11e/lens-query-lib/src/main/java/org/apache/lens/lib/query/HadoopFileFormatter.java
----------------------------------------------------------------------
diff --git a/lens-query-lib/src/main/java/org/apache/lens/lib/query/HadoopFileFormatter.java b/lens-query-lib/src/main/java/org/apache/lens/lib/query/HadoopFileFormatter.java
index 87aff69..d556fb0 100644
--- a/lens-query-lib/src/main/java/org/apache/lens/lib/query/HadoopFileFormatter.java
+++ b/lens-query-lib/src/main/java/org/apache/lens/lib/query/HadoopFileFormatter.java
@@ -135,7 +135,9 @@ public class HadoopFileFormatter extends AbstractFileFormatter {
    */
   @Override
   public void close() throws IOException {
-    rowWriter.close(Reporter.NULL);
+    if (null != rowWriter) {
+      rowWriter.close(Reporter.NULL);
+    }
   }
 
   @Override