diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/conf/InMemoryConfigurationStore.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/conf/InMemoryConfigurationStore.java index dc2c724aac7..07a876f87cb 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/conf/InMemoryConfigurationStore.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/conf/InMemoryConfigurationStore.java @@ -81,35 +81,63 @@ public long getConfigVersion() { return configVersion; } + /** + * Configuration mutations not logged (i.e. not persisted) but directly confirmed, as such, a list of persisted configuration mutations does not exist. + * @return null Configuration mutation list not applicable for this store. + */ @Override public List getConfirmedConfHistory(long fromId) { // Unimplemented. return null; } + /** + * Configuration mutations not logged (i.e. not persisted) but directly confirmed, as such, a list of persisted configuration mutations does not exist. + * @return null Configuration mutation list not applicable for this store. + */ @Override protected LinkedList getLogs() { // Unimplemented. return null; } + /** + * Configuration mutations applied directly in-memory, as such, there is no persistent configuration store. + * As there is no configuration store for versioning purposes, a conf store version is not applicable. + * @return null Conf store version not applicable for this store. + * @throws Exception + */ @Override public Version getConfStoreVersion() throws Exception { // Does nothing. return null; } + /** + * Configuration mutations not logged (i.e. not persisted), as such, they are not persisted and versioned. + * Hence, no version information to store. + * @throws Exception + */ @Override public void storeVersion() throws Exception { // Does nothing. } + /** + * Configuration mutations not logged (i.e. not persisted), as such, they are not persisted and versioned. + * Hence, a current version is not applicable. + * @return null A current version not applicable for this store. + */ @Override public Version getCurrentVersion() { // Does nothing. return null; } + /** + * Configuration mutations not logged (i.e. not persisted), as such, they are not persisted and versioned. + * Hence, version is always compatible, since it is in-memory. + */ @Override public void checkVersion() { // Does nothing. (Version is always compatible since it's in memory)