Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

polish SendMessageProcessor.preSend #3955

Closed
ferrirW opened this issue Mar 10, 2022 · 1 comment
Closed

polish SendMessageProcessor.preSend #3955

ferrirW opened this issue Mar 10, 2022 · 1 comment
Labels
type/code style

Comments

@ferrirW
Copy link
Contributor

ferrirW commented Mar 10, 2022 •
edited
Loading

rocketmq/broker/src/main/java/org/apache/rocketmq/broker/processor/SendMessageProcessor.java

Lines 704 to 710 in c840b20

response.setCode(-1);
super.msgCheck(ctx, requestHeader, response);
if (response.getCode() != -1) {
return response;
}
return response;

delete useless check
ferrirW pushed a commit to ferrirW/rocketmq that referenced this issue Mar 10, 2022
[ISSUE apache#3955] delete useless check
14710c9
Git-Yang pushed a commit that referenced this issue Mar 14, 2022
@Git-Yang
[ISSUE #3955] delete useless check
adbf3db
@Git-Yang
Copy link
Member

Merged

GenerousMan pushed a commit to GenerousMan/rocketmq that referenced this issue Aug 12, 2022
@Git-Yang
[ISSUE apache#3955] delete useless check
81eba49
carlvine500 pushed a commit to carlvine500/rocketmq-apache that referenced this issue Sep 10, 2024
@Git-Yang
[ISSUE apache#3955] delete useless check
69cdf8b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/code style
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Git-Yang @ferrirW