flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From walterddr <...@git.apache.org>
Subject [GitHub] flink pull request #5985: [FLINK-8135][docs] Add description to MessageParam...
Date Thu, 10 May 2018 21:06:23 GMT
Github user walterddr commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5985#discussion_r187460147
  
    --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/SubtaskIndexPathParameter.java
---
    @@ -44,4 +44,9 @@ protected String convertToString(final Integer value) {
     		return value.toString();
     	}
     
    +	@Override
    +	public String getDescription() {
    +		return "The index of a subtask. Integer value starts from 0, should be positive.";
    --- End diff --
    
    `must be non-negative`, if "0" is valid


---

Mime
View raw message