Skip to content

Commit c6216da

Browse files
committed
DPL: make new early forward optional
The new behavior breaks the TPC custom policy.
1 parent a498938 commit c6216da

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

Framework/Core/src/DataProcessingDevice.cxx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1054,7 +1054,11 @@ void DataProcessingDevice::fillContext(DataProcessorContext& context, DeviceCont
10541054
}
10551055

10561056
auto decideEarlyForward = [&context, &deviceContext, &spec, this]() -> ForwardPolicy {
1057-
ForwardPolicy defaultEarlyForwardPolicy = getenv("DPL_OLD_EARLY_FORWARD") ? ForwardPolicy::AtCompletionPolicySatisified : ForwardPolicy::AtInjection;
1057+
//ForwardPolicy defaultEarlyForwardPolicy = getenv("DPL_OLD_EARLY_FORWARD") ? ForwardPolicy::AtCompletionPolicySatisified : ForwardPolicy::AtInjection;
1058+
// Make the new policy optional until we handle some of the corner cases
1059+
// with custom policies which expect the early forward to happen only when
1060+
// all the data is available, like in the TPC case.
1061+
ForwardPolicy defaultEarlyForwardPolicy = getenv("DPL_NEW_EARLY_FORWARD") ? ForwardPolicy::AtInjection : ForwardPolicy::AtCompletionPolicySatisified;
10581062

10591063
/// We must make sure there is no optional
10601064
/// if we want to optimize the forwarding

0 commit comments

Comments
 (0)