[ https://issues.apache.org/jira/browse/PROTON-2140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16979605#comment-16979605 ]
ASF GitHub Bot commented on PROTON-2140: ---------------------------------------- astitcher commented on pull request #213: PROTON-2140: Lazy creation of various link related objects URL: https://github.com/apache/qpid-proton/pull/213#discussion_r349305058 ########## File path: c/src/core/codec.c ########## @@ -364,37 +364,72 @@ static int pn_data_inspect(void *obj, pn_string_t *dst) #define pn_data_hashcode NULL #define pn_data_compare NULL -pn_data_t *pn_data(size_t capacity) +pn_data_t *pni_data(size_t capacity, size_t initial_buffer) Review comment: Thinking further I'm reasonably convinced that always making it lazy shouldn't be an issue as we now make the buff only when we need it - so if we always make at least the previous 64 byte size then we do strictly no more work and if we delay the creation we can be avoid making a 64 byte buffer only to immediately expand it due to the size of the first interned data ---------------------------------------------------------------- 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: us...@infra.apache.org > proton-c has very high memory footprint for links > ------------------------------------------------- > > Key: PROTON-2140 > URL: https://issues.apache.org/jira/browse/PROTON-2140 > Project: Qpid Proton > Issue Type: Improvement > Reporter: Gordon Sim > Priority: Major > -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org