[ 
https://issues.apache.org/jira/browse/NIFI-11743?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Simon Bence updated NIFI-11743:
-------------------------------
    Description: 
The current definition of FlowRegistryClient allows implementations to return 
with a null value in cases such as list of buckets, flows or versions. In 
practice, this will not happen, but future implementations of changes has this 
option now.

This is something we want to eliminate both because it is considered as a bad 
pattern and because NiFi expect a valid (but maybe empty) return value. From 
code perspective this will not cause any change, but the document will be 
extended accordingly and will communicate that null is not a valid return value.

Even if this is not a code change and the current registry client behaves in 
this manner, it is considered as a breaking change thus this will be part of 
the 2.x line only.

  was:
The current definition of FlowRegisttryClient allows implementations to return 
with a null value in cases such as list of buckets, flows or versions. In 
practice, this will not happen, but future implementations of changes has this 
option now.

This is something we want to eliminate both because it is considered as a bad 
pattern and because NiFi expect a valid (but maybe empty) return value. From 
code perspective this will not cause any change, but the document will be 
extended accordingly and will communicate that null is not a valid return value.

Even if this is not a code change and the current registry client behaves in 
this manner, it is considered as a breaking change thus this will be part of 
the 2.x line only.


> Restrict FlowRegistryClient API to not allow null returns
> ---------------------------------------------------------
>
>                 Key: NIFI-11743
>                 URL: https://issues.apache.org/jira/browse/NIFI-11743
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework
>            Reporter: Simon Bence
>            Assignee: Simon Bence
>            Priority: Minor
>
> The current definition of FlowRegistryClient allows implementations to return 
> with a null value in cases such as list of buckets, flows or versions. In 
> practice, this will not happen, but future implementations of changes has 
> this option now.
> This is something we want to eliminate both because it is considered as a bad 
> pattern and because NiFi expect a valid (but maybe empty) return value. From 
> code perspective this will not cause any change, but the document will be 
> extended accordingly and will communicate that null is not a valid return 
> value.
> Even if this is not a code change and the current registry client behaves in 
> this manner, it is considered as a breaking change thus this will be part of 
> the 2.x line only.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to