-
Story
-
Resolution: Done
-
Medium
-
None
Additional performance requirement for existing batch-read
NCMP shall limit 50 cmHandles per operation
NCMP shall reject any request with over 50 cmHandles for any operation (number of operations is unlimited)
Issues & Decisions
- Batch support multiple operations with multiple cm handles each!
ecsakoc agreed to reject request if ANY of the operations attempt to read more then 50 cm handles (confirmed 8 Apr by ecsakoc ) - Agree HTTP Status code
ecsakoc agreed to use 413 Payload Too Large - Developer issue: in what layer to handle/throw exception: REST controller OR service Layer ?!
A/C
- Agree HTTP Error code when limit exceeded: 413 Payload Too Large (Confirmed by ecsakoc )
- Update documentation
- Demo to team and stakeholders
Out-of-scope
- Configurable limit, NCMP Hard-code the this limit as agreed with ecsakoc
How
- Introduce new (NCMP specific exception) e.g. OperationTooLargeException
- throw exception in org.onap.cps.ncmp.rest.controller.handlers.NcmpPassthroughResourceRequestHandler#validateDataOperationRequest (preferred)
- handle in NetworkCmProxyRestExceptionHandler.java and return 413
- documentation updates! (in the Yaml)