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.

Sign up for GitHub

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

feat(proxy): add metrics and trace for remoting send and pull RPC #844

Merged
merged 1 commit into from
Dec 19, 2023

feat(proxy): add metrics and trace for remoting send and pull RPC

0d76801
Select commit
Loading
Failed to load commit list.
Merged

feat(proxy): add metrics and trace for remoting send and pull RPC #844

feat(proxy): add metrics and trace for remoting send and pull RPC
0d76801
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/patch succeeded Dec 19, 2023 in 0s

26.56% of diff hit (within 50.00% threshold of 55.49%)

View this Pull Request on Codecov

26.56% of diff hit (within 50.00% threshold of 55.49%)

Annotations

Check warning on line 108 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java#L105-L108 

Added lines #L105 - L108 were not covered by tests

Check warning on line 123 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java#L114-L123 

Added lines #L114 - L123 were not covered by tests

Check warning on line 260 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java#L259-L260 

Added lines #L259 - L260 were not covered by tests

Check warning on line 267 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendPullMessageActivity.java#L267 

Added line #L267 was not covered by tests

Check warning on line 96 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java#L87-L96 

Added lines #L87 - L96 were not covered by tests

Check warning on line 157 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java#L157 

Added line #L157 was not covered by tests

Check warning on line 176 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java#L167-L176 

Added lines #L167 - L176 were not covered by tests

Check warning on line 187 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java#L187 

Added line #L187 was not covered by tests

Check warning on line 213 in proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/remoting/activity/ExtendSendMessageActivity.java#L210-L213 

Added lines #L210 - L213 were not covered by tests

Check warning on line 834 in proxy/src/main/java/com/automq/rocketmq/proxy/service/MessageServiceImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

proxy/src/main/java/com/automq/rocketmq/proxy/service/MessageServiceImpl.java#L834 

Added line #L834 was not covered by tests

Check warning on line 184 in store/src/main/java/com/automq/rocketmq/store/MessageStoreImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

store/src/main/java/com/automq/rocketmq/store/MessageStoreImpl.java#L184 

Added line #L184 was not covered by tests

Check warning on line 186 in store/src/main/java/com/automq/rocketmq/store/MessageStoreImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

store/src/main/java/com/automq/rocketmq/store/MessageStoreImpl.java#L186 

Added line #L186 was not covered by tests
View more details on Codecov