Skip to content

PHPC-1893: Implement SDAM event classes #1287

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jan 5, 2022

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Dec 30, 2021

@jmikola jmikola force-pushed the phpc-1893 branch 2 times, most recently from 1bc4625 to 8e34aee Compare January 5, 2022 18:58
@jmikola jmikola force-pushed the feature/sdam-monitoring branch from b5d33b5 to fa0715d Compare January 5, 2022 19:56
tanlisu and others added 6 commits January 5, 2022 14:57
Use phongo_objectid_init in getTopologyId() methods. Report topologyId as an ObjectId in debug info. Report TopologyDescription servers as ServerDescription in debug info.

Copy parameters in phongo_serverdescription_init and phongo_topologydescription_init for consistency with other functions (e.g. phongo_readconcern_init). This also creates phongo_serverdescription_init_ex for when we want to skip copying in Server::getServerDescription().

PHPC-1953: Add tests for debug handlers

PHPC-1959: Utilize Manager::addSubscriber() in tests

Skip TopologyClosedEvent test (pending PHPC-2023)
@jmikola jmikola marked this pull request as ready for review January 5, 2022 21:30
@jmikola jmikola merged this pull request into mongodb:feature/sdam-monitoring Jan 5, 2022
@jmikola jmikola deleted the phpc-1893 branch January 5, 2022 21:31
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.

2 participants