Issue 1:
Weekly Report of Dubbo

This is a weekly report of Dubbo. It summarizes what have changed in the project during the passed week, including pr merged, new contributors, and more things in the future.

It is all done by @dubbo-bot which is a collaborate robot.

Repo Overview

Basic data

Baisc data shows how the watch, star, fork and contributors count changed in the passed week.

Watch
Star
Fork
Contributors

3240
27450 (↑138)
15448 (↑112)
207 (↑3)

Issues & PRs

Issues & PRs show the new/closed issues/pull requests count in the passed week.

New Issues
Closed Issues
New PR
Merged PR

23
18
24
19

PR Overview

Thanks to contributions from community, Dubbo team merged 19 pull requests in the repository last week. They are:

fix #4406 (#4408)

Polish /#4348 : [ISSUE] Nacos Dubbo Registry can't sense the change (#4392)

Polish /#3695 : @reference field can't refer its' @servic… (#4385)

Observation of performance degradation when running under JDK 8 (#4381)

duplicate decrease for ExecuteLimitFilter onError (#4380)

AsyncRpcResult should handle exception when registering callback. (#4379)

fix qos configuration cannot work. #4377 (#4378)

format file: pom.xml of bom module (#4376)

Spring package scan compatibility, support both alibaba Service and apache Service. (#4375)

[Dubbo-4370] Add more data to serviceMetaData (#4372)

fix zipkin compatible. #3728 (#4368)

unify metadata and configcenter implementations (#4360)

fix empty protocol in consul registry. #4294 (#4354)

fix empty protocol for nacos. #4294. (#4349)

synchronized local variables or parameters should be set to final (#4325)

fix bug about nacos (#4308)

[Dubbo-4299]Fix NPE when PojoUtils realize Null element in collection(#4299) (#4300)

allow @service and @reference to merge attributes form annotations in lower levels of the annotation hierachy. (#4078)

[No issue] Fix: @method default value will be set to the field of MethodConfig (#3642)

Code Review Statistics

Dubbo encourages everyone to participant in code review, in order to improve software quality. Every week @dubbo-bot would automatically help to count pull request reviews of single github user as the following. So, try to help review code in this project.

Contributor ID
Pull Request Reviews

@ralf0131
8

@cvictory
7

@danielamorais
6

@chickenlj
4

@zonghaishang
3

@Leishunyu
2

@nullcodeexecutor
1

@moriadry
1

@Braavos96
1

@rolandhe
1

@owen200008
1

@zhaixiaoxiang
1

@guohao
1

@CrazyHZM
1

@dhchao11
1

@mercyblitz
1

Contributors Overview

It is Dubbo team's great honor to have new contributors from community. We really appreciate your contributions. Feel free to tell us if you have any opinion and please share this open source project with more people if you could. If you hope to be a contributor as well, please start from https://github.com/apache/incubator-dubbo/blob/master/CONTRIBUTING.md .

Here is the list of new contributors:

@RojerAlone

@dhchao11

@HzjNeverStop

Thanks to you all.

Note: This robot is supported by Collabobot.

Issue 2:
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

Travis

Steps to reproduce this issue

Create a PR.

See the build of travis.

see https://travis-ci.org/github/apache/dubbo/jobs/674706306

Expected Result

Passed.

Actual Result

Failed:

[ERROR] testContainer Time elapsed: 0.127 s <<< ERROR!
java.lang.IllegalStateException: Recursive update
at org.apache.dubbo.container.log4j.Log4jContainerTest.testContainer(Log4jContainerTest.java:31)

Travis always failed.

But sometimes it will passed after force-pushed without any change.

Compare:

https://travis-ci.org/github/apache/dubbo/jobs/674706306

https://travis-ci.org/github/apache/dubbo/builds/674993938