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.
---