Skip to content

Comments

Add notification flow and shutdown handling in SDK#393

Merged
ko80 merged 2 commits intoOpenVisualCloud:mainfrom
ko80:sdk-api-zero-copy-prep
May 26, 2025
Merged

Add notification flow and shutdown handling in SDK#393
ko80 merged 2 commits intoOpenVisualCloud:mainfrom
ko80:sdk-api-zero-copy-prep

Conversation

@ko80
Copy link
Collaborator

@ko80 ko80 commented May 16, 2025

  • Improve closed connection handling in the FFmpeg MCM plugin.
  • Add SDK client registry in Media Proxy.
  • Add event broker in Media Proxy.
  • Add SDK client shutdown notification flow.
  • Improve local connection manager shutdown handling in Media Proxy.
  • Modify connection activation flow to return the link status and
    return only when the link to multipoint group is confirmed.
  • Add SDK client registration flow in Media Proxy.
  • Add global context and shutdown flow in SDK.

@ko80 ko80 force-pushed the sdk-api-zero-copy-prep branch 2 times, most recently from 16309fb to 7738842 Compare May 23, 2025 17:45
@ko80 ko80 changed the title [DRAFT! DO NOT MERGE] Add notification flow and shutdown handling in SDK Add notification flow and shutdown handling in SDK May 23, 2025
@ko80 ko80 force-pushed the sdk-api-zero-copy-prep branch from 7738842 to 7ef4259 Compare May 23, 2025 17:48
@ko80 ko80 marked this pull request as ready for review May 23, 2025 18:51
@ko80 ko80 requested review from moleksy and soopel as code owners May 23, 2025 18:51
@ko80 ko80 requested review from tszumski and removed request for soopel May 23, 2025 18:51
* Improve closed connection handling in the FFmpeg MCM plugin.
* Add SDK client registry in Media Proxy.
* Add event broker in Media Proxy.
* Add SDK client shutdown notification flow.
* Improve local connection manager shutdown handling in Media Proxy.
* Modify connection activation flow to return the link status and
return only when the link to multipoint group is confirmed.
* Add SDK client registration flow in Media Proxy.
* Add global context and shutdown flow in SDK.

Signed-off-by: Konstantin Ilichev <konstantin.ilichev@intel.com>
@ko80 ko80 force-pushed the sdk-api-zero-copy-prep branch from 7ef4259 to 6039b36 Compare May 26, 2025 08:12
Signed-off-by: Konstantin Ilichev <konstantin.ilichev@intel.com>
Copy link
Collaborator

@moleksy moleksy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, great work!

@ko80
Copy link
Collaborator Author

ko80 commented May 26, 2025

LGTM, great work!

Thanks! More to come.

@ko80 ko80 merged commit b7dcb20 into OpenVisualCloud:main May 26, 2025
9 checks passed
ko80 added a commit that referenced this pull request May 28, 2025
* Add notification flow and shutdown handling in SDK
* Improve closed connection handling in the FFmpeg MCM plugin.
* Add SDK client registry in Media Proxy.
* Add event broker in Media Proxy.
* Add SDK client shutdown notification flow.
* Improve local connection manager shutdown handling in Media Proxy.
* Modify connection activation flow to return the link status and
return only when the link to multipoint group is confirmed.
* Add SDK client registration flow in Media Proxy.
* Add global context and shutdown flow in SDK.

Signed-off-by: Konstantin Ilichev <konstantin.ilichev@intel.com>
ko80 added a commit to ko80/Media-Communications-Mesh that referenced this pull request May 28, 2025
…lCloud#393)"

The commit is cherry-picked to the zero-copy branch as part of
a separate feature development.

This reverts commit b7dcb20.
ko80 added a commit that referenced this pull request May 28, 2025
)

The commit is cherry-picked to the zero-copy branch as part of
a separate feature development.

This reverts commit b7dcb20.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants