Skip to content

Fix getColonSeparatedKey does not work in 'URL simplified' mode.#5465

Merged
chickenlj merged 1 commit into2.7.5-releasefrom
simplified-url-interface
Dec 12, 2019
Merged

Fix getColonSeparatedKey does not work in 'URL simplified' mode.#5465
chickenlj merged 1 commit into2.7.5-releasefrom
simplified-url-interface

Conversation

@chickenlj
Copy link
Copy Markdown
Contributor

@chickenlj chickenlj commented Dec 12, 2019

Fixes #4621

   public String getColonSeparatedKey() {
        StringBuilder serviceNameBuilder = new StringBuilder();
        serviceNameBuilder.append(this.getServiceInterface());
        append(serviceNameBuilder, VERSION_KEY, false);
        append(serviceNameBuilder, GROUP_KEY, false);
        return serviceNameBuilder.toString();
    }

@chickenlj chickenlj changed the base branch from master to 2.7.5-release December 12, 2019 03:44
@chickenlj
Copy link
Copy Markdown
Contributor Author

Unstable ut failed the ci, merge directly.

@chickenlj chickenlj merged commit 6fc6494 into 2.7.5-release Dec 12, 2019
mercyblitz added a commit that referenced this pull request Dec 13, 2019
@chickenlj chickenlj deleted the simplified-url-interface branch December 13, 2019 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant