Use generic invocation via API , lost "version" value

[z529192557]

• I have searched the issues of this repository and believe that this is not a duplicate.
• I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

• Dubbo version: 2.7.3
• Operating System version: MAC OX 10.14.5
• Java version: 1.8

Steps to reproduce this issue

1. Use GenericService by API, assign the version is 1.0.0.test.qa.st
   

2. breakpoint at DubboInvoker.doInvoke

3. check the DubboInvoker.version = 0.0.0
   

Pls. provide [GitHub address] to reproduce this issue.

Expected Result

In the instance of DubboInvoker，the version should be 1.0.0.test.qa.st

Actual Result

In the instance of DubboInvoker，the version’s value actully is default value,0.0.0

Cause

this method : "ClusterUtils.mergeUrl" at line 87 ,remove the version key, so lost the version's value

[z529192557]

same as "group", "ClusterUtils.mergeUrl" at line 84, remove the group key , so lost the group value

[tswstarplanet]

Would you please submit a pr to fix it ?

[z529192557]

Would you please submit a pr to fix it ?
OK，I submit a pr to fix it，

[z529192557]

the modify in the issues #4525 leads to this problem.
I just fix GROUP_KEY and VERSION_KEY，

• when consumer has theirs value，and provider dose not have， use the consumer's value
• when provider has theirs value， use the provider's value

[beiwei30]

@z529192557 but I tried it on main branch, but failed to reproduce.

[beiwei30]

@z529192557 but I tried it on main branch, but failed to reproduce.

I didn't realize your test is direct invocation, pls. ignore my previous comment.

[z529192557]

@z529192557 but I tried it on main branch, but failed to reproduce.

I didn't realize your test is direct invocation, pls. ignore my previous comment.

Yes，it‘s direct invocation，only in direct invocation via API， can reproduce.