WorkflowAction should use Configurer style API
Description
Release Notes
blocks
Activity

Andreas Neumann September 18, 2015 at 4:36 AMEdited
Turns out that the existing code is broken.
it does not call the proper configure() method when executing program actions.
it does not set the class name in the action spec if the action implements the WorkflowAction interface directly.
Both of these work if the action extends AbstractWorkflowAction but overrides the legacy configure() method. It does not work if it overrides the new configure() method. Also does not work if the action directly implements WorkflowAction.
I will try to fix these issues also here.

Andreas Neumann September 17, 2015 at 9:30 PM

Andreas Neumann September 17, 2015 at 9:28 PM
None of the guides configures a workflow action.

Andreas Neumann September 17, 2015 at 8:52 PM
Changing all internal uses (tests, examples, etl-apps) to use new style. Documentation does not explain the configure() method. I think that is ok, because it is so similar to other configurers.

Gokul Gunasekaran September 17, 2015 at 7:53 PM
Yes, since we have the existing configure() method, there was no other way. We can clean it up when we remove the deprecated API in 3.4.
Details
Details
Assignee

Reporter

Currently it uses builder style API. Should change it to configurer style similar to programs.