lemonzone2010 commented on issue #531: can't get code and errorMsg in SendMessageContext when running SendMessageProcessor.executeSendMessageHookAfter URL: https://github.com/apache/rocketmq/issues/531#issuecomment-436883706 > I checked the code, found > `if (sendOK) {` > ` //xxxxxx` > ` doResponse(ctx, request, response);` > ` //xxxxxx` > ` return null;` > `}` > method doResponse had already processed the response. > In addition, feature SendMessageHook seems to be deprecated, i found no where use this hook in broker. - The method doResponse doesn't affect the SendMessageHook running. - I added some SendMessageHook in broker for tracing message,then i found this issue. Close it if you're sure SendMessageHook will be deprecated
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
