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

Posted to dev@celix.apache.org by "pnoltes (via GitHub)" <gi...@apache.org> on 2023/03/14 18:57:10 UTC

[GitHub] [celix] pnoltes commented on pull request #494: Remove buffer mechanism from websocket pubsub admin, make similar to tcp admin

pnoltes commented on PR #494:
URL: https://github.com/apache/celix/pull/494#issuecomment-1468659092

   > > The strategy in this fix is making it similar to the tcp admin.
   > 
   > I just noticed psa-tcp has similar problems. Then I'm OK merging this PR in its current form.
   > 
   > But let's hear what @pnoltes says about `pubsub_subscriber_t`'s `init`, which can not be used to tweak receiver thread attributes any more. psa-tcp has already broken this design.
   
   I agree, this is already broken so out of scope for this PR.


-- 
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: dev-unsubscribe@celix.apache.org

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