simonbence commented on code in PR #7218:
URL: https://github.com/apache/nifi/pull/7218#discussion_r1218085936
##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/registry/flow/FlowRegistryClientNode.java:
##########
@@ -44,7 +44,7 @@ public interface FlowRegistryClientNode extends ComponentNode
{
RegisteredFlow getFlow(FlowRegistryClientUserContext context, String
bucketId, String flowId) throws FlowRegistryException, IOException;
Set<RegisteredFlow> getFlows(FlowRegistryClientUserContext context, String
bucketId) throws FlowRegistryException, IOException;
- RegisteredFlowSnapshot getFlowContents(FlowRegistryClientUserContext
context, String bucketId, String flowId, int version, boolean fetchRemoteFlows)
throws FlowRegistryException, IOException;
+ FlowSnapshotContainer getFlowContents(FlowRegistryClientUserContext
context, String bucketId, String flowId, int version, boolean fetchRemoteFlows)
throws FlowRegistryException, IOException;
Review Comment:
This looks to be a breaking change on the API. I think, this can be okay for
a 2.X change, but the Apache ticket is marked as an 1.21.0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
