Gaffer icon indicating copy to clipboard operation
Gaffer copied to clipboard

FederatedOperationHandler Retains information from previous operation.

Open GCHQDev404 opened this issue 1 year ago • 2 comments

Describe the bug The FederatedOperationHandler has retains a field called graphs and context which persists for the life of the Handler within the FederatedStore, retaining information from the previous execution.

Expected behaviour Handler should not retain state from the previous execution.

GCHQDev404 avatar Nov 14 '23 12:11 GCHQDev404

Same issue happens with the context field within this class. But that was introduced earlier.

GCHQDev404 avatar Nov 15 '23 12:11 GCHQDev404

A potential solution is to just pass the graphs in to getAllGraphResults and mergeResults as parameters, and not keep them around at all between runs.

t92549 avatar Nov 15 '23 14:11 t92549