Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2634#discussion_r183051697
--- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/Worker.java
---
@@ -305,8 +305,9 @@ private void setupFlushTupleTimer(final Map<String,
Object> topologyConf, final
}
private void setupBackPressureCheckTimer(final Map<String, Object>
topologyConf) {
- final Integer workerCount =
ObjectReader.getInt(topologyConf.get(Config.TOPOLOGY_WORKERS));
- if (workerCount <= 1) {
+ Set<Integer> nonLocalTasks =
Sets.difference(workerState.getTaskToComponent().keySet(),
--- End diff --
`Sets.difference()` is already a named function and it's almost one line of
code. I think it's OK to just use it in this way?
---
