Skip to content
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

Implemented health api #116

Merged
merged 4 commits into from
Jan 6, 2025
Merged

Implemented health api #116

merged 4 commits into from
Jan 6, 2025

Conversation

rohanhcumca
Copy link
Contributor

Implemented report_partition_health and report_replica_health api's. Internally calling ReportPartitionHealth and ReportReplicaHealth respectively.

Copy link
Contributor

@OscarTHZhang OscarTHZhang left a comment

Choose a reason for hiding this comment

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

Please fix formatting and reply to agree with CLA Agreement

@rohanhcumca
Copy link
Contributor Author

rohanhcumca commented Jan 6, 2025 via email

@rohanhcumca
Copy link
Contributor Author

rohanhcumca commented Jan 6, 2025 via email

@rohanhcumca
Copy link
Contributor Author

Please fix formatting and reply to agree with CLA Agreement

done

Copy link
Collaborator

@youyuanwu youyuanwu left a comment

Choose a reason for hiding this comment

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

LGTM

@youyuanwu youyuanwu merged commit d9e9492 into Azure:main Jan 6, 2025
4 checks passed
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