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

Jump to bottom

Align with Dubbo: Route based on tag #593

Closed
flycash opened this issue Jun 11, 2020 · 3 comments · Fixed by #665
Closed

Align with Dubbo: Route based on tag #593

flycash opened this issue Jun 11, 2020 · 3 comments · Fixed by #665
Labels
help wanted Extra attention is needed occupied Issue has been occupied
Milestone
1.5.1

Comments

@flycash
Copy link
Member

flycash commented Jun 11, 2020 •
edited
Loading

What would you like to be added:

Provider could be tagged and then some invocation will be sent to providers with special tags

Why is this needed:

Dubbo support this but we don't
@flycash flycash added this to the 1.5.1 milestone Jun 11, 2020
@flycash flycash added the help wanted Extra attention is needed label Jun 11, 2020
@flycash
Copy link
Member Author

flycash commented Jun 11, 2020

It should be one of the Conditional Route implementations.

@relaxedCat
Copy link
Contributor

take it~

@watermelo
Copy link
Member

got it~

@zouyx zouyx added the occupied Issue has been occupied label Jul 2, 2020
@watermelo watermelo mentioned this issue Jul 20, 2020
Merged
@zouyx zouyx linked a pull request Jul 28, 2020 that will close this issue
Ftr: dynamic tag router #665
Merged
@zouyx zouyx closed this as completed Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed occupied Issue has been occupied
Projects
None yet
Milestone
1.5.1
Development

Successfully merging a pull request may close this issue.

Ftr: dynamic tag router watermelo/dubbo-go
5 participants
@zouyx @flycash @watermelo @fangyincheng @relaxedCat