Add high-level workflow describe#2326
Conversation
da59a8d to
de9320c
Compare
| public class WorkflowExecutionDescription extends WorkflowExecutionMetadata { | ||
| private final @Nonnull DescribeWorkflowExecutionResponse response; | ||
|
|
||
| public WorkflowExecutionDescription( |
There was a problem hiding this comment.
Can we make this constructor package private like its super class? I assume this is public because it is interceptable? Not sure the describe call needs to be interceptable if the list call isn't. Or if we want to make the list call interceptable, we can and then expose the constructor of WorkflowExecutionMetadata.
There was a problem hiding this comment.
Yes it is public because it is intercept-able, It was a requirement on the issue. The list call is not intercept-able right now but it makes sense to open an issue to add that as well and expose the constructor of WorkflowExecutionMetadata, but I would prefer to do that in a separate issue.
There was a problem hiding this comment.
Opened an issue to track making list intercept-able #2328
de9320c to
d3a8623
Compare
closes #2265