ruanwenjun commented on a change in pull request #6618:
URL: https://github.com/apache/dolphinscheduler/pull/6618#discussion_r738288162
##########
File path:
dolphinscheduler-common/src/main/java/org/apache/dolphinscheduler/common/utils/JSONUtils.java
##########
@@ -212,26 +208,28 @@ public static String findValue(JsonNode jsonNode, String
fieldName) {
}
/**
- * json to map
- * {@link #toMap(String, Class, Class)}
+ * json to map {@link #toMap(String, Class, Class)}
*
* @param json json
* @return json to map
*/
public static Map<String, String> toMap(String json) {
- return parseObject(json, new TypeReference<Map<String, String>>() {});
+ return parseObject(json, new TypeReference<Map<String, String>>() {
+ });
}
/**
* from the key-value generated json to get the str value no matter the
real type of value
- * @param json the json str
+ *
+ * @param json the json str
* @param nodeName key
* @return the str value of key
*/
public static String getNodeString(String json, String nodeName) {
Review comment:
Sorry, I see.
Your implantation is ok for me. Please revert the code format, it seems your
format is not better than current.
You can import the checkstyle in your IDE to help format your code.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
