HPCC-11650 Fix issues, with conditional varying exections paths

Conditional activities in subgraphs, that were re-executed
in particular if re-executed in master (e.g. in LOOP), where the
condition varied per execution could cause problems.
The code to detect the active sinks in a created subgraph,
unintentionally considered  activities that were not part
of the execution path, but had been in previous execution as
sinks.
This causes the master to call methods in the inactive activities,
e.g. onStart(). You may normally get away with it, but if the
onStart() relied on a value, created in a dependent subgraph that
had not been executed in this LOOP iteration, then it would fail.

This commit fixes the issue, by ensuring that only active sinks
are explicitly added to the 'activeSinks' list, each time the
subgraph is connected up.

Signed-off-by: Jake Smith <jake.smith@lexisnexis.com>
Merge pull request #6051 from jakesmith/hpcc-11650

HPCC-11650 Fix issues, with conditional varying exections paths

Reviewed-by: Gavin Halliday <ghalliday@hpccsystems.com>