[ 
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

Reply via email to