-
Notifications
You must be signed in to change notification settings - Fork 159
Performance[MQB]: Inline Put, Push, Ack, Confirm #833
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
base: main
Are you sure you want to change the base?
Conversation
5d71ba5 to
19e4a3d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build 2898 of commit 19e4a3d has completed with FAILURE
19e4a3d to
1ee45ff
Compare
1ee45ff to
afd7c70
Compare
Signed-off-by: dorjesinpo <[email protected]>
afd7c70 to
a87485a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build 3130 of commit a87485a has completed with FAILURE
| inline void ClusterNodeStats::onEvent<ClusterNodeStats::EventType::e_PUT>( | ||
| bsls::Types::Int64 value) | ||
| { | ||
| d_statContext_mp->adjustValue(ClusterNodeStatsIndex::e_STAT_PUT, value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Stats
| /// finished. | ||
| mqbi::QueueHandle* | ||
| getHandle(const bsl::shared_ptr<mqbi::QueueHandleRequesterContext>& | ||
| getHandle(const mqbi::OpenQueueConfirmationCookieSp& context, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Type change
| /// Stat context dedicated to this domain, to use as the parent stat | ||
| /// context for any queue in this domain. | ||
| StatContextMp d_statContext_mp; | ||
| const bsl::shared_ptr<bmqst::StatContext> d_statContext_sp; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Type change
duplicate of #177