Removed redundant timeout handling for executeCommand 86/122886/1
authorJulien Fontaine <julien.fontaine@bell.ca>
Tue, 27 Jul 2021 16:43:24 +0000 (12:43 -0400)
committerJulien Fontaine <julien.fontaine@bell.ca>
Tue, 27 Jul 2021 16:56:17 +0000 (12:56 -0400)
commit82e396d6917519468376d177dd6a2710e84fa23a
treedd6313e31e18a7a072e28494b351b503b559ecb7
parent3fb5e3aa70f26a7cc39b0ee756c97d4490d0bd94
Removed redundant timeout handling for executeCommand

executeCommand was called from a GlobalScope.async coroutine to handle executeCommand timeout.
Execution timeout is already handled on the gRPC side, there's no need to handle this timeout by wrapping the gRPC call with a coroutine timeout.
When a lot of long running processes in executeCommand were running, it was causing process to queue on BP side to get their executeCommand executed because all the capacity of couroutines were being used by some long running process.
This was causing a delay on the execution of prepareEnv and executeCommand and would even some time make those process timeout because of component timeout.

Issue-ID: CCSDK-3386
Signed-off-by: Julien Fontaine <julien.fontaine@bell.ca>
Change-Id: If3aef4b6b65006e874525436bf3f6d1b1a3d8c9a
ms/blueprintsprocessor/functions/python-executor/src/main/kotlin/org/onap/ccsdk/cds/blueprintsprocessor/functions/python/executor/ComponentRemotePythonExecutor.kt