BenjaminPerryRoss commented on a change in pull request #4818:
URL: https://github.com/apache/geode/pull/4818#discussion_r412493711
##########
File path:
geode-gfsh/src/main/java/org/apache/geode/management/internal/cli/domain/DataCommandResult.java
##########
@@ -381,6 +397,12 @@ public ResultModel toResultModel() {
}
ResultModel result = new ResultModel();
+
+ if (warningMessage != null && !warningMessage.isEmpty()) {
+ InfoResultModel info = result.addInfo(HEADER_INFO_SECTION);
Review comment:
I'm reluctant to change the addInfo to setHeader because I want the
structure of the output to match remove as closely as possible so I opted to
change the name to WARNING_INFO_SECTION as this section isn't used in any other
context.
----------------------------------------------------------------
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:
us...@infra.apache.org
