You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2020/06/17 02:20:32 UTC

[GitHub] [rocketmq] wangshaojie4039 opened a new issue #2102: send message with empty Body cause NullPointerException

wangshaojie4039 opened a new issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102


   The issue tracker is **ONLY** used for bug report(feature request need to follow [RIP process](https://github.com/apache/rocketmq/wiki/RocketMQ-Improvement-Proposal)). Keep in mind, please check whether there is an existing same report before your raise a new one.
   
   Alternately (especially if your communication is not a bug report), you can send mail to our [mailing lists](http://rocketmq.apache.org/about/contact/). We welcome any friendly suggestions, bug fixes, collaboration and other improvements.
   
   Please ensure that your bug report is clear and that it is complete. Otherwise, we may be unable to understand it or to reproduce it, either of which would prevent us from fixing the bug. We strongly recommend the report(bug report or feature request) could include some hints as the following:
   
   **BUG REPORT**
   
   1. when client receive empty body message from broker,consumeService throw NullPointerException
   
   - What did you do (The steps to reproduce)?
   
   - What did you expect to see?
   
   - What did you see instead?
   
   ![image](https://user-images.githubusercontent.com/22910636/84847630-2029bf80-b084-11ea-983b-66ae88b62c6f.png)
   
   
   
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] coder-zzzz commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
coder-zzzz commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645228834


   @Jaskey maybe he use other client instead of java client


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] Jaskey commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
Jaskey commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645226882


   @wangshaojie4039  how do you produce this message to producer? Since the producer api should check if the body is null


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] Jaskey commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
Jaskey commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645331407


   @zongtanghu Thx, would you please help to review the bug fix pr https://github.com/apache/rocketmq/pull/2104 ?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] wangshaojie4039 commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
wangshaojie4039 commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645124471


   > Hi, can you please provide more information on how to reproduce this problem, including
   > 
   > * which version rocketmq-client/rocketmq you are using (because we need to locate which line caused the problem, but different version has different line numbers)
   > * ...
   > 
   > Thanks so much and we will follow up~
   
   rocketmq version is 4.5.1


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] zongtanghu commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
zongtanghu commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645167953


   I think it's necessary to add the empty Body limination logical codes for messages on broker. @ShannonDing @wangshaojie4039 @RongtongJin 


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] zongtanghu commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
zongtanghu commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645249500


   @Jaskey  In java sdk of rocketmq, it check the message body exactly.But our members use other language rocektmq sdk, such as rocektmq-go sdk. I also put forward a issue. https://github.com/apache/rocketmq-client-go/issues/501


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] imaffe commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
imaffe commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645116798


   Hi, can you please provide more information on how to reproduce this problem, including
   
   - which version rocketmq-client/rocketmq you are using (because we need to locate which line caused the problem, but different version has different line numbers)
   - ...
   
   Thanks so much and we will follow up~
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [rocketmq] coder-zzzz commented on issue #2102: send message with empty Body cause NullPointerException

Posted by GitBox <gi...@apache.org>.
coder-zzzz commented on issue #2102:
URL: https://github.com/apache/rocketmq/issues/2102#issuecomment-645180265


   IMO,empty body should be allowed, sometimes mq just used to notify clients about sth change with no message.
   I think should fix the client's problem instead of add limit on broker


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org