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.

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

LABEL in could be any string #3424

Merged
merged 2 commits into from
Dec 13, 2021
Merged

Conversation

liwenhui-soul
Copy link
Contributor

@liwenhui-soul liwenhui-soul commented Dec 7, 2021

normally, string like "zone127.0.0.1_111" is illegal for name_label, this pr is to make these illegal strings in `` could be legal for name_label.

@czpmango
Copy link
Contributor

czpmango commented Dec 8, 2021

Please add some test cases.

@darionyaphet
Copy link
Contributor

darionyaphet commented Dec 8, 2021

SO, this PR can replace name_label with STRING?

@liwenhui-soul
Copy link
Contributor Author

liwenhui-soul commented Dec 8, 2021

SO, this PR can replace name_label with STRING?

replace STRING with name_label

@liwenhui-soul
Copy link
Contributor Author

Please add some test cases.

done

@liwenhui-soul liwenhui-soul mentioned this pull request Dec 10, 2021
7 tasks
Copy link
Contributor

@critical27 critical27 left a comment

Choose a reason for hiding this comment

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

Well done

@critical27 critical27 merged commit 1c2e1aa into vesoft-inc:master Dec 13, 2021
@yixinglu yixinglu added the doc affected PR: improvements or additions to documentation label Dec 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc affected PR: improvements or additions to documentation ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants