celix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [celix] pnoltes commented on pull request #249: Add env vars for bind/connect urls in pubsub
Date Wed, 03 Jun 2020 17:22:31 GMT

pnoltes commented on pull request #249:
URL: https://github.com/apache/celix/pull/249#issuecomment-638338893


   > @pnoltes the OSX error is because pubsub_utils.c:186 tries to put 512 kb on the stack,
but the default stack size on OSX is 512 kb, leading to problems. I can either reduce the
stack size, use a memory allocation or increase the stack size for threads create in celix.
All have their downsides:
   > 
   > ## reduce stack size
   > There is no specified maximum for topic/scope, so any reduction in stack size implicitly
means that topic/scope gain a limit.
   > 
   > ## use memory allocation
   > Slow
   > 
   > ## increase stack size
   > Works for things celix controls, but as soon as someone uses the `pubsub_getEnvironmentVariableWithScopeTopic`
function in a user-created thread on a system with small stack sizes, this error comes up
again.
   
   use memory allocation. reading env is part of the setup for pubsub and IMO for setup performance
is nice to have. 


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



Mime
View raw message