You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "moleske (GitHub)" <gi...@apache.org> on 2019/02/12 01:35:22 UTC

[GitHub] [geode] moleske commented on pull request #3177: GEODE-6033: Support dynamic VMs in DistributedDiskDirRule

Just because I want to double check, is there need for `toArray`?  I probably would have used for each with the list unless there's a significant performance bonus

```java
List<VM> allVMs = getAllVMs();
allVMs.add(getController());
for (VM vm : allVMs) {
    ...
}
```

[ Full content available at: https://github.com/apache/geode/pull/3177 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org