Page Comparison

...

#	Issue	Notes/Jira	Decision
1	There are 2 separate query objects defined in OpenAPI		Combine them into one 'Conditions' has the better structure but 'cmHandleQueryRestParameters' is a better name
2	Exact format of 'legacy' Conditions unclear and maybe not generic enough	Study demo: https://wikilf-onap.onapatlassian.orgnet/wiki/download/attachments/9300039216450241/CPS644.mkv?version=1&modificationDate=1636566033000&api=v2
3	Definition of 'Conditions' is not Generic enough	$ref: '#/components/schemas/ModuleNamesAsJsonArray Should be something like '#/components/ConditionParametersNameValuePairs
4	Agree backward incompatibility	Discuss with stakeholders

...

#	Layer	Proposed Changes
1	REST Layer	openapi.yaml change /v1/data/ch/searches to /v1/ch/id-searches ncmp.yaml change queryCmHandles method to executeCmHandleIdSearch Use the CmHandleQueryRestParameters schema to both of them components.yaml merge Conditions schema to CmHandleQueryRestParameters schema delete Conditions schema
2	Controller Layer	NetworkCmProxyController Update controller methods to use the new schema Both component method call separated service method Rename queryCmHandles method to executeCmHandleIdSearch
3	Service Layer	NetworkCmProxyDataService Create new method for query cmHandles Keep queryCmHandles method and rename it to queryCmHandleIds Both method should be calling separate methods in CpsDataService Validation logic should be moved to an separated method and call it from both method CpsDataService Update queryCmHandles method return type to Set<CmHandles> Use separate methods to CmHandlesId search to just return a collection of CmHandleIds (extract from DataNode Objects) CpsAdminPersistenceService Update queryCmHandles will always return complet DataNode object(s). No updates can be done
4	Persistence Layer	ModuleReferenceQuery Update queryCmHandles method return type to Set<CmHandles> Update queryCmHandles method to handle both filtering parameters Create to separete method for CmHandlesId searches with new filtering parameters (use the original queryCmHandles method code)

...

Versions Compared