You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/12/15 03:20:51 UTC

[GitHub] [pulsar] tongsucn commented on pull request #13293: [Issue 13285][pulsar-client] Optimize MessageBuilder and SharedBuffer to avoid unnecessary memory copy.

tongsucn commented on pull request #13293:
URL: https://github.com/apache/pulsar/pull/13293#issuecomment-994254501


   > `auto msg = MessageBuilder().setContent(value).build();`
   
   @BewareMyPower  I think the first `setContent` assertion should be fine. Because it internally allocates a same-sized `std::string` and calls `std::copy` explicitly, rather than using `std::string`'s copy constructor.


-- 
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: commits-unsubscribe@pulsar.apache.org

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