You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by we...@apache.org on 2020/04/17 04:39:35 UTC
[hadoop] branch branch-2.10 updated: HDFS-15283. Cache pool MAXTTL
is not persisted and restored on cluster restart. Contributed by Stephen
O'Donnell.
This is an automated email from the ASF dual-hosted git repository.
weichiu pushed a commit to branch branch-2.10
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-2.10 by this push:
new 9ec141c HDFS-15283. Cache pool MAXTTL is not persisted and restored on cluster restart. Contributed by Stephen O'Donnell.
9ec141c is described below
commit 9ec141c77fad0f02c84ddfb5b6e6489c7189b77b
Author: Stephen O'Donnell <so...@apache.org>
AuthorDate: Thu Apr 16 20:16:41 2020 -0700
HDFS-15283. Cache pool MAXTTL is not persisted and restored on cluster restart. Contributed by Stephen O'Donnell.
Signed-off-by: Wei-Chiu Chuang <we...@apache.org>
(cherry picked from commit 3481895f8a9ea9f6e217a0ba158c48da89b3faf2)
(cherry picked from commit aaad947c740c91c36139f3f0569ae78b53bca682)
(cherry picked from commit 041c93a26cd0ff705bea9d6283951702cf2cf1c2)
(cherry picked from commit 0ea8f3a19cd0b841dda6ba5c4e19887d9fc9cbb4)
---
.../java/org/apache/hadoop/hdfs/server/namenode/CacheManager.java | 8 ++++++++
.../apache/hadoop/hdfs/server/namenode/TestCacheDirectives.java | 6 +++++-
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/CacheManager.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/CacheManager.java
index 6726202..3fe90b4 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/CacheManager.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/CacheManager.java
@@ -1041,6 +1041,10 @@ public class CacheManager {
if (p.getLimit() != null)
b.setLimit(p.getLimit());
+ if (p.getMaxRelativeExpiryMs() != null) {
+ b.setMaxRelativeExpiry(p.getMaxRelativeExpiryMs());
+ }
+
pools.add(b.build());
}
@@ -1106,6 +1110,10 @@ public class CacheManager {
if (p.hasLimit())
info.setLimit(p.getLimit());
+ if (p.hasMaxRelativeExpiry()) {
+ info.setMaxRelativeExpiryMs(p.getMaxRelativeExpiry());
+ }
+
addCachePool(info);
}
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestCacheDirectives.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestCacheDirectives.java
index 92928d1..4ab9c89 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestCacheDirectives.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestCacheDirectives.java
@@ -568,10 +568,12 @@ public class TestCacheDirectives {
String groupName = "partygroup";
FsPermission mode = new FsPermission((short)0777);
long limit = 747;
+ long maxExpiry = 1234567890;
dfs.addCachePool(new CachePoolInfo(pool)
.setGroupName(groupName)
.setMode(mode)
- .setLimit(limit));
+ .setLimit(limit)
+ .setMaxRelativeExpiryMs(maxExpiry));
RemoteIterator<CachePoolEntry> pit = dfs.listCachePools();
assertTrue("No cache pools found", pit.hasNext());
CachePoolInfo info = pit.next().getInfo();
@@ -579,6 +581,7 @@ public class TestCacheDirectives {
assertEquals(groupName, info.getGroupName());
assertEquals(mode, info.getMode());
assertEquals(limit, (long)info.getLimit());
+ assertEquals(maxExpiry, (long)info.getMaxRelativeExpiryMs());
assertFalse("Unexpected # of cache pools found", pit.hasNext());
// Create some cache entries
@@ -639,6 +642,7 @@ public class TestCacheDirectives {
assertEquals(groupName, info.getGroupName());
assertEquals(mode, info.getMode());
assertEquals(limit, (long)info.getLimit());
+ assertEquals(maxExpiry, (long)info.getMaxRelativeExpiryMs());
assertFalse("Unexpected # of cache pools found", pit.hasNext());
dit = dfs.listCacheDirectives(null);
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org