Re: Travis CI failed

2018-09-12 Thread Jerrick Zhu
It failed after add cache. https://travis-ci.org/apache/incubator-dubbo/jobs/428004974 On Thu, Sep 13, 2018 at 1:27 PM Jerrick Zhu wrote: > OK, I'll take it. > > On Thu, Sep 13, 2018 at 12:52 PM YunKun Huang wrote: > >> I don't understand why cache mvn repo files will cause a compile issue.

Re: Travis CI failed

2018-09-12 Thread Jerrick Zhu
OK, I'll take it. On Thu, Sep 13, 2018 at 12:52 PM YunKun Huang wrote: > I don't understand why cache mvn repo files will cause a compile issue. > Refer to travis document ( > https://docs.travis-ci.com/user/caching/#things-not-to-cache), > ``` > The cache’s purpose is to make installing

Re: Issue all of the random failed UT cases

2018-09-12 Thread Jerrick Zhu
I'm working on them now. I mean record the failed cases whenever you meet. On Thu, Sep 13, 2018 at 1:06 PM Ian Luo wrote: > We should fix these issues Jerrick filed. > > On Thu, Sep 13, 2018 at 11:23 AM Jerrick Zhu wrote: > > > ping > > > > On Wed, Sep 12, 2018 at 10:09 AM Jerrick Zhu wrote:

Re: Issue all of the random failed UT cases

2018-09-12 Thread Ian Luo
We should fix these issues Jerrick filed. On Thu, Sep 13, 2018 at 11:23 AM Jerrick Zhu wrote: > ping > > On Wed, Sep 12, 2018 at 10:09 AM Jerrick Zhu wrote: > > > Hi, community > > > > I found that many cases failed on travis, randomly. > > > >

Re: Travis CI failed

2018-09-12 Thread YunKun Huang
I don't understand why cache mvn repo files will cause a compile issue. Refer to travis document (https://docs.travis-ci.com/user/caching/#things-not-to-cache), ``` The cache’s purpose is to make installing language-specific dependencies easy and fast, so everything related to tools like

Re: Issue all of the random failed UT cases

2018-09-12 Thread Jerrick Zhu
ping On Wed, Sep 12, 2018 at 10:09 AM Jerrick Zhu wrote: > Hi, community > > I found that many cases failed on travis, randomly. > > https://github.com/apache/incubator-dubbo/issues/2492 > https://github.com/apache/incubator-dubbo/issues/2490 >

Re: Travis CI failed

2018-09-12 Thread Jerrick Zhu
On Wed, Sep 12, 2018 at 10:15 AM Ian Luo wrote: > I suggest to add back maven cache configuration to check if it's recovered. > It helps on speeding the build. > I prefer stable than speed. How about others' thought? > > On Wed, Sep 12, 2018 at 8:47 AM Jerrick Zhu wrote: > > > When I enabled

Re: Local forced trigger mock does not take effect

2018-09-12 Thread kirito
Hi,Ian.luo and community I has debugged for a while and probably understand the workflow of mock in dubbo, what confused me was that > I know dubbo was intended to implement `fail:`,`force:`,`throw` these > functions(from the workflow of dubbo source code), but I can’t find the > description

Re: [Discuss]Prepare for the next release v2.6.4

2018-09-12 Thread Jerrick Zhu
On Wed, Sep 12, 2018 at 9:44 AM Huxing Zhang wrote: > On Mon, Sep 10, 2018 at 10:35 AM Jun Liu wrote: > > > > > Do you think we need another one to be the release manager? In case > > > that we can make sure our release guide works perfectly. > > > > Yes, I think we need a new manager to do the

Re: Local forced trigger mock does not take effect

2018-09-12 Thread Ian Luo
Jingfeng, Further more, is there a need to provide support for > `fail:`,`force:`,`throw`? What the behavior they should own? > I think 'fail:', 'force:' should work, pls. check the code here [1]. What unexpected behavior did you see? I have not tried it out, but it looks we have the

Re: [DISCUSS] A metric to measure the response time to issues/prs

2018-09-12 Thread yuhang xiu
Hi, Maintaining dubbo is not the main job of many people, so there may be a delay in processing the pr. In addition, I have a pr that has been placed for more than 30 days, but I don't want it to be merged as soon as possible, but I hope it will be reviewed and recognized by more people. As

Re: about grpc support

2018-09-12 Thread Jason Joo
hi, dp According to principle of gRPC which is a solution crossing front-end(mobile) to back-end(server) using HTTP/2, i think it's suitable to be integrated in Controller layer(some like dubbo-consumer) but not service layer(dubbo-provider). Nothing blocks us to use gRPC in consumers, right?

Re: 关于dubbo的版本

2018-09-12 Thread Huxing Zhang
Hi, There is an FAQ item related to this: https://github.com/apache/incubator-dubbo/blob/master/FAQ.md#which-version-should-i-choose On Wed, Sep 12, 2018 at 11:06 AM 宗荣 wrote: > >

Re: [DISCUSS] A metric to measure the response time to issues/prs

2018-09-12 Thread kirito
Hi With this topic, I ask a question. Look me as normal dubbo user What’s more can I do to speed up the merge of these pr? A mail to the mailing list? @ the member? Like this pull request, https://github.com/apache/incubator-dubbo/pull/2418

Re: [DISCUSS] A metric to measure the response time to issues/prs

2018-09-12 Thread Ian Luo
Let's do it in simple way. Pls. put 'issue resolution' and 'open issues' icons straightforward on the head of the README.md. Let's face it and don't be shame. On Wed, Sep 12, 2018 at 3:44 PM Jerrick Zhu wrote: > On Wed, Sep 12, 2018 at 12:53 PM Huxing Zhang wrote: > > > Hi, > > > > I just

Re: [ANNOUNCE] Welcome new committer Minxuan Zhuang

2018-09-12 Thread Ian Luo
Congratulations! On Wed, Sep 12, 2018 at 1:29 PM Jerrick Zhu wrote: > Congrats. > > On Wed, Sep 12, 2018 at 11:44 AM Zonghai Shang wrote: > > > Congratulations! > > > > yiji > > > > yuhang xiu 于2018年9月12日周三 上午10:09写道: > > > > > Congratulations! > > > Welcome to Dubbo. :) > > > > > > Huxing

Re: [DISCUSS] A metric to measure the response time to issues/prs

2018-09-12 Thread Jerrick Zhu
On Wed, Sep 12, 2018 at 12:53 PM Huxing Zhang wrote: > Hi, > > I just find an article [1] complaining about the response time to pull > request and issues. > That's true. Our issue/PR's response time is too long. > > I think we can use a metric to measure the response time, and I found this >

Re: Remaining branches

2018-09-12 Thread jun liu
* 2.6.3-release, merged. I have just merged 2.6.3-release into 2.6.x, and it’s been removed. * apache-release-script, deleted. The changes in this branch are only limited to release.sh, which is only a release script for Apache Release purpose. I found it also has been merged to 2.6.x, so I

Re: Local forced trigger mock does not take effect

2018-09-12 Thread kirito
Hi I have a try right now,it was very strange. Works very well, but I think the blank is a illegal character too. As exception says : > invalid mock=“" contain illegal character, only digit, letter, '-', '_' > and '.' is legal. And I search for the dubbo source code find that in

[Dubbo benchmark ]The test results are mixed with the dubbo log.

2018-09-12 Thread Zonghai Shang
Hi , I am using performance testing to find out that the output is confusing, it would be better if the output is supported to the file. dubbo-benchmark: https://github.com/dubbo/dubbo-benchmark Here is my test output: ``` # Fork: 1 of 1 # Warmup Iteration 1: Sep 12, 2018 2:22:31 PM