Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

feat(new_metrics): implement the metric & its prototype #1075

Merged

Conversation

empiredan
Copy link
Contributor

Copy link
Member

@acelyc111 acelyc111 left a comment

Choose a reason for hiding this comment

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

LGTM

inline To down_cast(From *from)
{
// Perform a compile-time assertion to check whether <From> class is derived from <To> class.
static_assert(std::is_base_of<typename std::remove_pointer<From>::type,
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks great to me :) 👍

Copy link
Member

@acelyc111 acelyc111 left a comment

Choose a reason for hiding this comment

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

LGTM

@levy5307 levy5307 merged commit 0d632e3 into XiaoMi:prometheus-dev Mar 18, 2022
acelyc111 pushed a commit to acelyc111/rdsn that referenced this pull request Jun 17, 2022
empiredan added a commit to empiredan/rdsn that referenced this pull request Jun 17, 2022
empiredan added a commit to empiredan/rdsn that referenced this pull request Jun 20, 2022
empiredan added a commit to empiredan/rdsn that referenced this pull request Jun 20, 2022
acelyc111 pushed a commit to acelyc111/rdsn that referenced this pull request Jun 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants