search

meetecho / janus-gateway

Janus WebRTC Server
https://janus.conf.meetecho.com
GNU General Public License v3.0
8.25k stars 2.48k forks source link

feat: An attempt to forward a handle's media stream with multiple threads for videoroom. #3323

Closed wswaaa123 closed 9 months ago

wswaaa123 commented 9 months ago

Hi, this PR tries to add multithreaded packet forwarding support for videoroom.

I found that when a meeting is distributed by multiple people, if a handle forwards multiple media streams in the same thread, the "Discarding too old outgoing packet" situation will occur, then the whole server stucks. I wonder if this problem can be tackled with multiple threads forwarding a single handle's media stream on sufficient server resources. So this PR is an attempt to forward a handle's media stream with multiple threads.

januscla commented 9 months ago

Thanks for your contribution, @wswaaa123! Please make sure you sign our CLA, as it's a required step before we can merge this.

lminiero commented 9 months ago

@wswaaa123 have you checked #3067? That's exactly what that patch does. Please test and provide feedback to that.

lminiero commented 9 months ago

Closing as a duplicate of an already existing PR. If you believe your patch provides anything that one doesn't already, please provide more info.