xuzhenbao opened a new issue #396:
URL: https://github.com/apache/celix/issues/396


   In the `importRegistration_proxyFunc` and `remoteServiceAdmin_send` 
functions, they only pass the CURL error to the service caller, but the remote 
service method invocation error is ignored. I think we should return both the 
CURL error and the remote service method invocation error to the service method 
caller. And in doing so, to avoid conflicts between CURL error codes and remote 
service method error codes, we can map the CURL errors and the remote service 
method invocation errors to the CELIX error codes. In the remote service 
client, we only handle the CELIX error code. For error code mapping, we can 
make the following definition:
   ~~~C
   #define CELIX_CURL_START_ERROR (CELIX_START_USERERR + 1000)
   #define CELIX_CURL_ERROR(curl_code) (CELIX_CURL_START_ERROR + (curl_code))
   
   #define CELIX_USER_SERVICE_START_ERROR (CELIX_START_USERERR + 2000)
   #define CELIX_USER_SERVICE_ERROR(user_service_code) 
(CELIX_USER_SERVICE_START_ERROR + (user_service_code))
   ~~~
   


-- 
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]


Reply via email to