Skip to content

Conversation

@trustin
Copy link
Contributor

@trustin trustin commented Feb 8, 2020

As requested by @wu-sheng 😉

@mrproliu mrproliu added agent Language agent related. plugin Plugin for agent or collector. Be used to extend the capabilities of default implementor. test Test requirements about performance, feature or before release. labels Feb 8, 2020
@mrproliu mrproliu requested a review from wu-sheng February 8, 2020 12:18
@mrproliu mrproliu added this to the 7.0.0 milestone Feb 8, 2020
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

Waiting for auto tests only.

Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

FYI @kezhenxu94 In the latest 0.98 Armeria test, seems the context propagation breaks. Could you recheck?

Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

Latest 0.98 Armeria can't pass the test.

Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

Request @kezhenxu94 recheck the plugin

@wu-sheng wu-sheng removed this from the 7.0.0 milestone Feb 8, 2020
@kezhenxu94 kezhenxu94 added this to the 7.0.0 milestone Feb 14, 2020
@kezhenxu94 kezhenxu94 requested a review from wu-sheng February 15, 2020 04:10
Copy link
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks 😄

@kezhenxu94 kezhenxu94 dismissed wu-sheng’s stale review February 15, 2020 04:26

Requested changes are addressed

@kezhenxu94 kezhenxu94 merged commit 5fed153 into apache:master Feb 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent Language agent related. plugin Plugin for agent or collector. Be used to extend the capabilities of default implementor. test Test requirements about performance, feature or before release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants