[GitHub] [hadoop] bharatviswa504 commented on a change in pull request #1247: HDDS-1926. The new caching layer is used for old OM requests but not updated.

[GitBox <gi...@apache.org>]

bharatviswa504 commented on a change in pull request #1247: HDDS-1926. The new caching layer is used for old OM requests but not updated.
URL: https://github.com/apache/hadoop/pull/1247#discussion_r311761847
 
 

 ##########
 File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OmMetadataManagerImpl.java
 ##########
 @@ -269,14 +276,23 @@ protected void initializeOmTables() throws IOException {
     userTable =
         this.store.getTable(USER_TABLE, String.class, VolumeList.class);
     checkTableStatus(userTable, USER_TABLE);
+
+    // As now we have eviction policies, and for non-HA code path we don't
+    // support cache and cleanup policies setting cache to manual.
+    TableCacheImpl.CacheCleanupPolicy cleanupPolicy;
+    if (isRatisEnabled) {
+      cleanupPolicy = TableCacheImpl.CacheCleanupPolicy.NEVER;
 
 Review comment:
   Done.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org