From 50dcc335be3fd1bdf8400775556eb491a445a370 Mon Sep 17 00:00:00 2001
From: Tsz-wo Sze <szetszwo@apache.org>
Date: Thu, 18 Feb 2010 23:22:52 +0000
Subject: [PATCH] HADOOP-6558. Return null in HarFileSystem.getFileChecksum(..)
 since no checksum algorithm is implemented.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@911646 13f79535-47bb-0310-9956-ffa450edef68
---
 CHANGES.txt                                               | 3 +++
 src/java/org/apache/hadoop/fs/HarFileSystem.java          | 7 +++++++
 src/test/core/org/apache/hadoop/fs/TestHarFileSystem.java | 7 +++++++
 3 files changed, 17 insertions(+)

diff --git a/CHANGES.txt b/CHANGES.txt
index 7354c4cec0..e94da20df7 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -210,6 +210,9 @@ Trunk (unreleased changes)
     HADOOP-6570. RPC#stopProxy throws NPE if getProxyEngine(proxy) returns
     null. (hairong)
 
+    HADOOP-6558. Return null in HarFileSystem.getFileChecksum(..) since no
+    checksum algorithm is implemented.  (szetszwo)
+
 Release 0.21.0 - Unreleased
 
   INCOMPATIBLE CHANGES
diff --git a/src/java/org/apache/hadoop/fs/HarFileSystem.java b/src/java/org/apache/hadoop/fs/HarFileSystem.java
index 8350d46a9e..1b75e31073 100644
--- a/src/java/org/apache/hadoop/fs/HarFileSystem.java
+++ b/src/java/org/apache/hadoop/fs/HarFileSystem.java
@@ -539,6 +539,13 @@ public class HarFileSystem extends FilterFileSystem {
             makeRelative(this.uri.toString(), new Path(hstatus.name)));
   }
 
+  /**
+   * @return null since no checksum algorithm is implemented.
+   */
+  public FileChecksum getFileChecksum(Path f) {
+    return null;
+  }
+
   /**
    * Returns a har input stream which fakes end of 
    * file. It reads the index files to get the part 
diff --git a/src/test/core/org/apache/hadoop/fs/TestHarFileSystem.java b/src/test/core/org/apache/hadoop/fs/TestHarFileSystem.java
index cacc862448..1768f21690 100644
--- a/src/test/core/org/apache/hadoop/fs/TestHarFileSystem.java
+++ b/src/test/core/org/apache/hadoop/fs/TestHarFileSystem.java
@@ -45,4 +45,11 @@ public class TestHarFileSystem {
       e.printStackTrace(System.out);
     }
   }
+
+  @Test
+  public void testFileChecksum() {
+    final Path p = new Path("har://file-localhost/foo.har/file1");
+    final HarFileSystem harfs = new HarFileSystem();
+    Assert.assertEquals(null, harfs.getFileChecksum(p));
+  }
 }