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

Develop #3

Merged
merged 319 commits into from
Sep 11, 2020
Merged

Develop #3

merged 319 commits into from
Sep 11, 2020

Conversation

sdttttt
Copy link
Owner

@sdttttt sdttttt commented Sep 11, 2020

What this PR does:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

cityiron and others added 30 commits July 29, 2020 17:33
@cityiron
add comment for public method
08c3674
@watermelo
Add: add ip address match function
4ef21cb
@watermelo
Add: add listener for tag router
891ec6c
@watermelo
Merge branch 'develop' into featrue/tagRouter
0d56ab0
@watermelo
Mod: update tag router unit test
8580ad0
@AlexStocks
Merge pull request #670 from zouyx/hotfix/fix1.4
0686ca9 
Fix: apollo data change listener
@Patrick0308
format code
f2261ce
@cityiron
add code format and comment
6f77a0c
@cityiron
ugly code to elegant
ddf63de
Modify README_CN
9b73313
Modify README_CN.md
55c84cd
@AlexStocks
Merge pull request #676 from Patrick0308/rest_methodtype
db1e5da 
Fix: GoRestServer can't deploy the HTTP interfaces with same path and different method type
@AlexStocks
Merge pull request #683 from QMHTMY/master
abcc4df 
Mod: 用词错误，忝列的意思是有愧于排列其中，应该是添加或者添列。
@zouyx
refactor config center
2e96585
@AlexStocks
Modify README_CN
bc46c9a
@AlexStocks
Modify README_CN.md
12680fd
@watermelo
Add: add unit test for dynamic tag
70c01a5
@watermelo
Add: add zk jar for tag test
a31a4e2
@lzp0412
Merge pull request #18 from apache/develop
65b215a 
Develop
@zouyx
refactor config center
73e7bdd
@zouyx
add case for this feature
d5a491e
@zouyx
refactor config center
3e9f57a
@lzp0412
unregister instance when registry destroy
0ef7765
@zouyx
fix review comment
d32cbdb
@zouyx
fix review comment
e0dd74e
@zouyx
fix review comment
8e6d2ac
@wenxuwan
try to fix zk problem
7ad26fd
@wenxuwan
try to fix ut failed
53587ec
@wenxuwan
change the default value to 15m
7a88bab
@wenxuwan
change the default value to 15min
c33f4ea
AlexStocks and others added 28 commits September 5, 2020 11:29
@AlexStocks
Merge pull request #722 from dubbo-x/read
6388c4a 
fix exporter append
@AlexStocks
Merge pull request #735 from wenxuwan/master
bb197cc 
Fix: #734
@skyao
improve map access concurrency
f11bc6d
@cvictory
Ftr: [refer dubbo 2.7.6] attachment type from map[string]stiring to m…
49c005a 
…ap[string]interface{} (#713)

* support attachment from map[sting]string to map[string]interface{} in invocation and result
@zouyx
Fix: integration testing problems
aa4d36a
@zouyx
Fix: integration testing problems
94fe6c6
@zouyx
Fix: integration testing problems
93cb0ac
@zouyx
Fix: integration testing problems
bff2cd6
@zouyx
Fix: integration testing problems
4cd2487
@beiwei30
address batch mode notification
a192291
@zouyx
Fix: integration testing problems
27a8dc7
@zouyx
Fix: integration testing problems
c3adeae
@zouyx
Fix: integration testing problems
a15d6af
@zouyx
Fix: integration testing problems
04b35ea
fix review comment
a76745f
@zouyx
delete useless file
9f50dc2
@beiwei30
enhance comments
a87b814
@Patrick0308
Merge pull request #740 from zouyx/feature/fixTestProblmes
837f41f 
Fix: integration testing problems
@zouyx
Revert "Fix: integration testing problems"
f146507
@beiwei30
move debug() into refreshInvokers and remove update method
6209b52
@skyao
Merge remote-tracking branch 'upstream/develop' into improve/map-acce…
99f2c3a 
…ss-concurreny
@fangyincheng
Merge pull request #744 from apache/revert-740-feature/fixTestProblmes
51f4bc3 
Revert "Fix: integration testing problems"
@skyao
Merge branch 'develop' into improve/map-access-concurreny
6299ce4
@zouyx
Merge pull request #739 from skyao/improve/map-access-concurreny
fe8e2f8 
Imp: Improve map access concurrency
@watermelo
Fix: rename SethealthChecker to SetHealthChecker
8ac1f91
@watermelo
Fix: syntax error
c9726e4
@zouyx
Merge pull request #741 from beiwei30/address-batch-mode
e7d38aa 
Address notification batch mode
@AlexStocks
Merge pull request #746 from watermelo/Fix/rename_health_checher
7bb924a 
Fix: rename SethealthChecker to SetHealthChecker
@sdttttt sdttttt merged commit 0bc4334 into sdttttt:sdttttt-patch-1 Sep 11, 2020
@pull-assistant
Copy link

pull-assistant bot commented Sep 11, 2020 •
edited
Loading

Score: 0.84

Best reviewed: commit by commit

Optimal code review plan (30 warnings, 6 commits squashed)

add comment for public method

...ter_impl/zone_aware_cluster.go 50% changes removed in add code format and ...

     Add: add ip address match function

     Add: add listener for tag router

Merge branch 'develop' into featrue/tagRouter

...ter/loadbalance/round_robin.go 75% changes removed in Merge pull request #...

...loadbalance/consistent_hash.go 67% changes removed in Merge pull request #...

registry/consul/utils_test.go 60% changes removed in Merge pull request #...

go.mod 56% changes removed in change module versio...

...parser/configuration_parser.go 55% changes removed in Merge pull request #...

remoting/consul/test_agent.go 50% changes removed in Merge pull request #...

remoting/zookeeper/client.go 50% changes removed in Merge pull request #...

     Mod: update tag router unit test

Merge pull request #670 from zouyx/hotfix/fix1.4

go.mod 50% changes removed in Mod: merge branch to...

     format code

add code format and comment

.../zone_aware_cluster_invoker.go 67% changes removed in ugly code to elegant

     ugly code to elegant

     Modify README_CN

     Modify README_CN.md

Merge pull request #676 from Patrick0308/rest_methodtype

...rver_impl/go_restful_server.go 92% changes removed in optimized code

     Merge pull request #683 from QMHTMY/master

refactor config center

config/base_config.go 90% changes removed in fix review comment

config/consumer_config.go 67% changes removed in Merge pull request #...

     Modify README_CN

     Modify README_CN.md

     Add: add unit test for dynamic tag

     Add: add zk jar for tag test

Merge pull request #18 from apache/develop

...rver_impl/go_restful_server.go 85% changes removed in use adapter model to...

refactor config center

config/consumer_config.go 71% changes removed in Merge remote-trackin...

     add case for this feature

     refactor config center

     unregister instance when registry destroy

fix review comment

config/base_config.go 67% changes removed in fix review comment

     fix review comment

     fix review comment

try to fix zk problem

config/registry_config.go 50% changes removed in Merge pull request #...

     try to fix ut failed

     change the default value to 15m

     change the default value to 15min

     fix review comment

Fix kubernetes ut ... FMT k8s pkg

Squashed 2 commits:

     fix review comment

register service instance after provider config load

registry/protocol/protocol.go 55% changes removed in modify name

     modify name

     fix review comment

     fix comments

     Merge pull request #693 from sxllwx/fix_k8s_ut

Merge pull request #692 from wenxuwan/master

config/registry_config.go 50% changes removed in Merge pull request #...

     Merge pull request #689 from lzp0412/develop

     fix travis

Merge pull request #665 from watermelo/featrue/tagRouter

.../router/tag/tag_router_test.go 95% changes removed in Revert "Ftr: dynamic...

     Revert "Ftr: dynamic tag router"

     fix travis

Merge pull request #3 from apache/develop

config/registry_config.go 50% changes removed in Merge pull request #...

common/constant/key.go 50% changes removed in Merge pull request #...

...rver_impl/go_restful_server.go 69% changes removed in use adapter model to...

...router/tag/router_rule_test.go 96% changes removed in Mod: merge branch to...

Merge branch 'develop' into zoneware

...uster/router/tag/tag_router.go 98% changes removed in Merge pull request #...

config/registry_config.go 50% changes removed in Merge remote-trackin...

.../router/tag/tag_router_test.go 95% changes removed in Merge pull request #...

...rver_impl/go_restful_server.go 62% changes removed in use adapter model to...

     stack level

     Merge pull request #687 from zouyx/feature/optimizeRemotesConfiguratio...

Merge pull request #696 from apache/revert-665-featrue/tagRouter

...router/tag/router_rule_test.go 56% changes removed in Mod: merge branch to...

add default priority... ... reduce cognitive in ...

Squashed 2 commits:

...er/directory/base_directory.go 80% changes removed in fix empty applicatio...

add cluster interceptor

...uster_impl/failover_cluster.go 50% changes removed in Merge remote-trackin...

a quick prototype for bitmap router chain

...condition/listenable_router.go 99% changes removed in make condition route...

...ter/router/condition/router.go 78% changes removed in make condition route...

...uster/router/tag/tag_router.go 48% changes removed in make condition route...

import split

...r_impl/base_cluster_invoker.go 50% changes removed in change format

cluster/cluster_interceptor.go 50% changes removed in change format

Add: add setInvoker function for router chain

...ster/router/tag/router_rule.go 43% changes removed in Mod: merge branch to...

.../router/tag/tag_router_test.go 50% changes removed in Mod: merge branch to...

     Mod: merge branch to resolve conflict

     replace bzr to git

     fix empty application configure always return true in base_directory s...

     added comment to file.go

     make condition router no-poolable and introduce consumer url in Regist...

Merge pull request #662 from williamfeng323/feature/application-level-...

...stdata/router_config_error.yml 83% changes removed in Merge remote-trackin...

...fig/testdata/router_config.yml 62% changes removed in Merge remote-trackin...

     Correct words and Format codes

     Format codes

     change format

Merge pull request #659 from DogBaoBao/zoneware

cluster/loadbalance/util.go 46% changes removed in Merge remote-trackin...

add tls support

config/protocol_config.go 75% changes removed in add tls support

     add nolint

     change module version

     upgrade getty version

fix getty const

config/consumer_config.go 50% changes removed in fix getty const

fix getty const

protocol/dubbo/config.go 80% changes removed in fix getty const

optimized code ... use adapter model to...

Squashed 2 commits:

     remove adapter

optimize header transmit in RestClient and RestServer

...col/rest/server/rest_server.go 75% changes removed in add some comments

add some comments

...col/rest/server/rest_server.go 71% changes removed in modify comments

...rver_impl/go_restful_server.go 55% changes removed in modify some comments...

     modify some comments and when parsing parameters occurred error, retur...

     modify comments

     remove new function

     modify some comments

     modify some comments and modify some logger's message

     optimized code

     let go_restful_server support same url and different methodType

     format code

     fix nacos unit test failed

     Merge remote-tracking branch 'remotes/dubbogo/develop' into develop

     update getty pkg

     Merge pull request #705 from lzp0412/develop

     add tls support

     change module version

     upgrade getty version

fix getty const

config/consumer_config.go 50% changes removed in fix getty const

     fix getty const

     rm go version

fix

go.mod 50% changes removed in rm go version

     Merge remote-tracking branch 'origin/AddTslSupport' into AddTslSupport

     Merge pull request #702 from dubbo-x/fix

     rm go version

     Merge pull request #704 from Howie66/master

Powered by Pull Assistant. Last update 08c3674 ... aaa8be3. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.