Shardingsphere: Optimize proxy instance definition in registry center

[menghaoranss]

According to the suggestions made by @dongzl and @kimmking in the issue #5627 , use ip+port instead of ip + threadId to define proxy instance in registry center is better.

/
├─orchestration-namespace
├─orchestration-name
│ ├─registry
│ │ ├─instances
│ │ │ ├─127.0.0.1@3307@xxx
│ │ │ ├─127.0.0.1@3308@xxx

[swayamraina]

Hi!
I believe OrchestrationInstance holds this data of ip + thread-id which needs to be changed.
I am not sure how to get the port on which the instance is running.

[menghaoranss]

@swayamraina For proxy，the port can be passed to ShardingOrchestrationFacade when proxy starts, and for jdbc, thread id is better.

[swayamraina]

@menghaoranss can you please review yhe PR for any improvements.

[menghaoranss]

@menghaoranss can you please review yhe PR for any improvements.

Of course, i will take the time to review, thanks for you contribution.

[kimmking]

@swayamraina Expecting your forward.

[menghaoranss]

@swayamraina any problems?

[swayamraina]

Will update the code as per review. Thanks!

[menghaoranss]

@swayamraina The pr #6187 you submitted has lasted too long, there may already be a lot of conflicts, can you please resubmit another pr based on our discussion?

[swayamraina]

sure @menghaoranss will sync the PR

[menghaoranss]

sure @menghaoranss will sync the PR

merged! thansk for your contribution.

[swayamraina]

Thanks, will try to contribute more :)