Github user roshannaik commented on a diff in the pull request:
https://github.com/apache/storm/pull/2634#discussion_r183200719
--- 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 --
I meant ... would be nice to have functions like isSingleWorkerMode() and
hasOutboundTasks() that hide the detail of whatever mechanics are being
employed to figure it out. I suspect they these functions will see more usage.
---
