sodonnel opened a new pull request #3161:
URL: https://github.com/apache/ozone/pull/3161
## What changes were proposed in this pull request?
In ListPipeline.java we have:
```
@Override
public void execute(ScmClient scmClient) throws IOException {
Stream<Pipeline> stream = scmClient.listPipelines().stream();
if (!Strings.isNullOrEmpty(factor)) {
stream = stream.filter(
p -> ReplicationConfig.getLegacyFactor(p.getReplicationConfig())
.toString().compareToIgnoreCase(factor) == 0);
}
if (!Strings.isNullOrEmpty(state)) {
stream = stream.filter(p -> p.getPipelineState().toString()
.compareToIgnoreCase(state) == 0);
}
stream.forEach(System.out::println);
}
```
This code will fail if the pipeline is an EC pipeline. We should also allow
for filtering on EC pipelines.
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6382
## How was this patch tested?
New unit tests and verified in docker cluster
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
