xiaoyuyao commented on a change in pull request #661: HDDS-976: Parse network
topology from yaml file
URL: https://github.com/apache/hadoop/pull/661#discussion_r272008223
##########
File path:
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/net/NodeSchemaLoader.java
##########
@@ -165,6 +169,81 @@ private NodeSchemaLoadResult loadSchema(File schemaFile)
throws
return schemaList;
}
+ /**
+ * Load user defined network layer schemas from a YAML configuration file.
+ * @param schemaFilePath path of schema file
+ * @return all valid node schemas defined in schema file
+ */
+ public NodeSchemaLoadResult loadSchemaFromYaml(String schemaFilePath)
+ throws IllegalArgumentException {
+ try {
+ File schemaFile = new File(schemaFilePath);
+ if (!schemaFile.exists()) {
+ String msg = "Network topology layer schema file " + schemaFilePath +
+ " is not found.";
+ LOG.warn(msg);
+ throw new IllegalArgumentException(msg);
+ }
+ return loadSchemaFromYaml(schemaFile);
+ } catch (Exception e) {
+ throw new IllegalArgumentException("Fail to load network topology node"
+ + " schema file: " + schemaFilePath + " , error:" +
e.getMessage());
+ }
+ }
+
+ /**
+ * Load network topology layer schemas from a YAML configuration file.
+ * @param schemaFile schema file
+ * @return all valid node schemas defined in schema file
+ * @throws ParserConfigurationException ParserConfigurationException happen
+ * @throws IOException no such schema file
+ * @throws SAXException xml file has some invalid elements
+ * @throws IllegalArgumentException xml file content is logically invalid
+ */
+ private NodeSchemaLoadResult loadSchemaFromYaml(File schemaFile) {
+ LOG.info("Loading network topology layer schema file " + schemaFile);
Review comment:
NIT: can we use parameterized log4j like below
LOG.info("Loading network topology layer schema file {}", schemaFile);
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
