[
https://issues.apache.org/jira/browse/FLINK-8906?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16396965#comment-16396965
]
ASF GitHub Bot commented on FLINK-8906:
---------------------------------------
Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/5671#discussion_r174139638
--- Diff:
flink-clients/src/main/java/org/apache/flink/client/program/rest/RestClusterClient.java
---
@@ -567,7 +567,7 @@ public GetClusterStatusResponse getClusterStatus() {
@Override
public int getMaxSlots() {
- return 0;
+ return -1;
--- End diff --
I suggest to assign this magic number to some `static final` variable with
a descriptive name and use it here and in `CliFrontend`, so that the connection
is clear.
> Flip6DefaultCLI is not tested in org.apache.flink.client.cli tests
> ------------------------------------------------------------------
>
> Key: FLINK-8906
> URL: https://issues.apache.org/jira/browse/FLINK-8906
> Project: Flink
> Issue Type: Bug
> Components: Client, Tests
> Affects Versions: 1.5.0, 1.6.0
> Reporter: Nico Kruber
> Assignee: Nico Kruber
> Priority: Blocker
> Labels: flip-6
> Fix For: 1.5.0, 1.6.0
>
>
> Various tests in {{org.apache.flink.client.cli}} only test with the
> {{DefaultCLI}} but should also test {{Flip6DefaultCLI}}.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
