[GitHub] [dubbo] LinShunKang commented on issue #5004: RpcUtils.getReturnTypes(Invocation) 性能问题

2019-09-18 Thread GitHub
@chickenlj 你好,我看了 `performance-tuning-2.7.x` 分支上相关代码,整体思路和我的差不多,但是,实现方式不太一样: * Dubbo 的实现方式是在生成每一个 `RpcInvocation` 对象的时候从缓存中获取 `returnTypes` 并放到 `RpcInvocation` 对象中保存起来 * 我的实现方式是直接修改 `RpcUtils.getReturnTypes(Invocation)` 的实现,从缓存中获 `returnTypes`并返回 个人观点: Dubbo 的实现方式会导致每一个 RpcInvocation 对象的占用空间变大,如果

[GitHub] [dubbo] 1076572358 opened issue #5081: 有人碰到dubbo跟rabbitmq消费者一起启动有冲突吗

2019-09-18 Thread GitHub
使用rebbitmq消费机打开后批量消费数据,dubbo消费者调用dubbo生产者就调不通了,超时。有人碰到过吗。 [ Full content available at: https://github.com/apache/dubbo/issues/5081 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscr

[GitHub] [dubbo] chickenlj commented on issue #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

2019-09-18 Thread GitHub
change format to `dubbo.application.parameters=[{a:1}, {b:2}]` [ Full content available at: https://github.com/apache/dubbo/pull/4899 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsu

[GitHub] [dubbo] 1076572358 commented on issue #5081: 有人碰到dubbo跟rabbitmq消费者一起启动有冲突吗

2019-09-18 Thread GitHub
spring.rabbitmq.listener.simple.concurrency = 3 ,这里开启的并发concurrency大于3就调用不到服务生产者这里来了 实在是百思不得其解 [ Full content available at: https://github.com/apache/dubbo/issues/5081 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org ---

[GitHub] [dubbo-go] codecov-io commented on issue #208: fix issue#207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo-go/pull/208?src=pr&el=h1) Report > Merging [#208](https://codecov.io/gh/apache/dubbo-go/pull/208?src=pr&el=desc) > into > [master](https://codecov.io/gh/apache/dubbo-go/commit/4cf8f0e6097b79d03980fe0cf3f2d8ba008173d6?src=pr&el=desc) > will **increas

[GitHub] [dubbo] mzorro commented on issue #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

2019-09-18 Thread GitHub
> change format to `dubbo.application.parameters=[{a:1}, {b:2}]` @chickenlj why do you think this format is more readable? It's not even standard JSON. [ Full content available at: https://github.com/apache/dubbo/pull/4899 ] This message was relayed via gitbox.apache.org for notifications@dubbo

[GitHub] [dubbo] cvictory closed pull request #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

2019-09-18 Thread GitHub
[ pull request closed by cvictory ] [ Full content available at: https://github.com/apache/dubbo/pull/4899 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificati

[GitHub] [dubbo] lxf-NewBanker commented on issue #1070: com.alibaba.dubbo.rpc.RpcException: Forbid consumer 169.254.226.207 access service com.base.service.AppService from registry localhost:2181 use

2019-09-18 Thread GitHub
@jkxqj https://github.com/apache/dubbo/issues/684 中,[david1228用户](https://github.com/david1228) 评论说: > 我的本地环境中也遇到类似的问题, 报错信息: Forbid consumer 10.58.95.10 access service com.xxx.mms.api.service.IAlbumService from registry 36.110.219.106:2181 use dubbo version 2.5.3, Please check registry acc

[GitHub] [dubbo] codecov-io commented on issue #5070: [Dubbo-4331] Improve api doc #4331

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo/pull/5070?src=pr&el=h1) Report > Merging [#5070](https://codecov.io/gh/apache/dubbo/pull/5070?src=pr&el=desc) > into > [master](https://codecov.io/gh/apache/dubbo/commit/b53624e4e88a545a3478525bac58aac3ac12afba?src=pr&el=desc) > will **decrease** co

[GitHub] [dubbo] codecov-io commented on issue #3850: Fix registry check=false invalid

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo/pull/3850?src=pr&el=h1) Report > Merging [#3850](https://codecov.io/gh/apache/dubbo/pull/3850?src=pr&el=desc) > into > [master](https://codecov.io/gh/apache/dubbo/commit/87c78878c639cfd0b055ed370d9daa4f4d9b1373?src=pr&el=desc) > will **increase** co

[GitHub] [dubbo] vio-lin opened issue #5082: ProtobufTypeBuilder should support generate type definition which contains Bytes List or Bytes Map

2019-09-18 Thread GitHub
- [x] I have searched the [issues](https://github.com/apache/dubbo/issues) of this repository and believe that this is not a duplicate. - [x] I have checked the [FAQ](https://github.com/apache/dubbo/blob/master/FAQ.md) of this repository and believe that this is not a duplicate. ### Environment

[GitHub] [dubbo] cvictory closed pull request #5079: fix set generic method error

2019-09-18 Thread GitHub
[ pull request closed by cvictory ] [ Full content available at: https://github.com/apache/dubbo/pull/5079 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificati

[GitHub] [dubbo] cvictory closed issue #3629: ServiceConfig duplicate export produces misleading exception

2019-09-18 Thread GitHub
[ issue closed by cvictory ] [ Full content available at: https://github.com/apache/dubbo/issues/3629 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-u

[GitHub] [dubbo] cvictory closed pull request #5078: delete 'config.' prefix for url generated from ConfigCenterConfig

2019-09-18 Thread GitHub
[ pull request closed by cvictory ] [ Full content available at: https://github.com/apache/dubbo/pull/5078 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificati

[GitHub] [dubbo] cvictory closed issue #5001: dubbo:config-center timeout配置不生效

2019-09-18 Thread GitHub
[ issue closed by cvictory ] [ Full content available at: https://github.com/apache/dubbo/issues/5001 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-u

[GitHub] [dubbo] voyagertanyao commented on issue #117: dubbo2.5.3注解方式无效

2019-09-18 Thread GitHub
@wangjunwei87 方案可行 [ Full content available at: https://github.com/apache/dubbo/issues/117 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@

[GitHub] [dubbo] htynkn commented on issue #5070: [Dubbo-4331] Improve api doc #4331

2019-09-18 Thread GitHub
What maven command should use with this change? I try `./mvnw clean javadoc:aggregate -Dmaven.javadoc.skip=false` somehow those changes can't apply to javadoc [ Full content available at: https://github.com/apache/dubbo/pull/5070 ] This message was relayed via gitbox.apache.org for notification

[GitHub] [dubbo-go] beiwei30 closed pull request #208: fix issue#207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
[ pull request closed by beiwei30 ] [ Full content available at: https://github.com/apache/dubbo-go/pull/208 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifica

[GitHub] [dubbo-go] beiwei30 opened pull request #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
**What this PR does**: **Which issue(s) this PR fixes**: Fixes #207 **Special notes for your reviewer**: **Does this PR introduce a user-facing change?**: ```release-note ``` [ Full content available at: https://github.com/apache/dubbo-go/pull/209 ] This message was relayed via gitbox.apa

[GitHub] [dubbo-go] wongoo commented on pull request #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
just return here [ Full content available at: https://github.com/apache/dubbo-go/pull/209 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@d

[GitHub] [dubbo-go] codecov-io commented on issue #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=h1) Report > Merging [#209](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=desc) > into > [develop](https://codecov.io/gh/apache/dubbo-go/commit/a977cc9975bd9041816fe87c3f26badbb732eecd?src=pr&el=desc) > will **decrea

[GitHub] [dubbo] codecov-io commented on issue #5069: [Dubbo 5052] Fix Executes take no effect when using generic call #5052

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo/pull/5069?src=pr&el=h1) Report > Merging [#5069](https://codecov.io/gh/apache/dubbo/pull/5069?src=pr&el=desc) > into > [master](https://codecov.io/gh/apache/dubbo/commit/a3c89e2e47bcb4c690d454a7c2a6eca712b64a75?src=pr&el=desc) > will **increase** co

[GitHub] [dubbo-go] codecov-io commented on issue #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=h1) Report > Merging [#209](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=desc) > into > [develop](https://codecov.io/gh/apache/dubbo-go/commit/a977cc9975bd9041816fe87c3f26badbb732eecd?src=pr&el=desc) > will **decrea

[GitHub] [dubbo-go] AlexStocks commented on pull request #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
pls delete this ref. thx. [ Full content available at: https://github.com/apache/dubbo-go/pull/209 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsu

[GitHub] [dubbo-website] realabbas opened issue #474: Poor Browser Leverage Caching

2019-09-18 Thread GitHub
I just ran a Page Speed test on http://dubbo.apache.org/ and found that it lacks browser caching. ![image](https://user-images.githubusercontent.com/43527087/65164446-8a142a80-da5a-11e9-8709-09bc86bbb7f0.png) ![image](https://user-images.githubusercontent.com/43527087/65164465-939d9280-da5a-11e9

[GitHub] [dubbo-go] codecov-io commented on issue #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=h1) Report > Merging [#209](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=desc) > into > [develop](https://codecov.io/gh/apache/dubbo-go/commit/a977cc9975bd9041816fe87c3f26badbb732eecd?src=pr&el=desc) > will **decrea

[GitHub] [dubbo-go] codecov-io commented on issue #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=h1) Report > Merging [#209](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=desc) > into > [develop](https://codecov.io/gh/apache/dubbo-go/commit/a977cc9975bd9041816fe87c3f26badbb732eecd?src=pr&el=desc) > will **decrea

[GitHub] [dubbo-go] codecov-io commented on issue #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
# [Codecov](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=h1) Report > Merging [#209](https://codecov.io/gh/apache/dubbo-go/pull/209?src=pr&el=desc) > into > [develop](https://codecov.io/gh/apache/dubbo-go/commit/a977cc9975bd9041816fe87c3f26badbb732eecd?src=pr&el=desc) > will **increa

[GitHub] [dubbo] zhaoxilingcheng commented on issue #346: zookeeper一直出现大量连接断开

2019-09-18 Thread GitHub
看你的业务逻辑处理是个长处理还是?一般接口3~5秒可以了 -- 原始邮件 -- 发件人: "machelcheng"; 发送时间: 2019年9月18日(星期三) 晚上10:48 收件人: "apache/dubbo"; 抄送: "得不到"<308870...@qq.com>; "Comment"; 主题: Re: [apache/dubbo] zookeeper一直出现大量连接断开 (#346) 原因是timeout设置时间太短,导致在验证时写入超时,所以要检查一下connect接口上的timeout,记

[GitHub] [dubbo] zlba111 closed issue #5068: 如何通过代码或者接口调用的方式获取dubbo服务的参数列表和参数类型?

2019-09-18 Thread GitHub
[ issue closed by zlba111 ] [ Full content available at: https://github.com/apache/dubbo/issues/5068 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-un

[GitHub] [dubbo-go] gaoxinge opened pull request #210: skip subscribe on provider side when using consul

2019-09-18 Thread GitHub
**What this PR does**: - skip subscribe on provider side when using consul **Which issue(s) this PR fixes**: Fixes # **Special notes for your reviewer**: - none **Does this PR introduce a user-facing change?**: ```release-note none ``` [ Full content available at: https://github.com/apach

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #131: fix emoji encode error

2019-09-18 Thread GitHub
@fangyincheng @pantianying @aliiohs @AlexStocks pls review [ Full content available at: https://github.com/apache/dubbo-go-hessian2/pull/131 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [dubbo-go] AlexStocks closed pull request #209: fix issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
[ pull request closed by AlexStocks ] [ Full content available at: https://github.com/apache/dubbo-go/pull/209 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifi

[GitHub] [dubbo] sonymoon commented on issue #5070: [Dubbo-4331] Improve api doc #4331

2019-09-18 Thread GitHub
> What maven command should use with this change? > I try `./mvnw clean javadoc:aggregate -Dmaven.javadoc.skip=false` somehow > those changes can't apply to javadoc `./mvnw clean javadoc:aggregate -Dmaven.javadoc.skip=false -Prelease` . `-P release` is required because the javadoc maven plugin

[GitHub] [dubbo-go] beiwei30 closed issue #207: common/utils/net_test.go:25 cannot pass

2019-09-18 Thread GitHub
[ issue closed by beiwei30 ] [ Full content available at: https://github.com/apache/dubbo-go/issues/207 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications

[GitHub] [dubbo-go]

2019-09-18 Thread GitHub
[ Full content available at: https://github.com/apache/dubbo-go/issues/103 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.or

[GitHub] [dubbo-go-hessian2] lovepoem commented on pull request #131: fix emoji encode error

2019-09-18 Thread GitHub
Please remove author [ Full content available at: https://github.com/apache/dubbo-go-hessian2/pull/131 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications

[GitHub] [dubbo-js] KeithZhang commented on issue #127: 不支持callback?

2019-09-18 Thread GitBox
KeithZhang commented on issue #127: 不支持callback? URL: https://github.com/apache/dubbo-js/issues/127#issuecomment-532946474 现在还不支持 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [dubbo] wayilau closed pull request #4746: Remove unnecessary extend.

2019-09-18 Thread GitHub
[ pull request closed by wayilau ] [ Full content available at: https://github.com/apache/dubbo/pull/4746 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificatio

[GitHub] [dubbo] cvictory closed pull request #5084: revert #5072 by logging error msg.

2019-09-18 Thread GitHub
[ pull request closed by cvictory ] [ Full content available at: https://github.com/apache/dubbo/pull/5084 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificati

[GitHub] [dubbo] CodingSinger opened issue #5085: 修复ZookeeperRegistry在监听所有接口的情况下,接口目录丢失导致的bug

2019-09-18 Thread GitHub
- [x] I have searched the [issues](https://github.com/apache/dubbo/issues) of this repository and believe that this is not a duplicate. - [x] I have checked the [FAQ](https://github.com/apache/dubbo/blob/master/FAQ.md) of this repository and believe that this is not a duplicate. ### Environment

[GitHub] [dubbo] CodingSinger opened pull request #5086: fix the bug can't subscribe the interface #5085

2019-09-18 Thread GitHub
fix the bug #5085 [ Full content available at: https://github.com/apache/dubbo/pull/5086 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@d

[GitHub] [dubbo] htynkn commented on issue #5071: Revert 1 2.7.2 release

2019-09-18 Thread GitHub
please provide more detail for your PR. I'm going to close this, feel free to reopen it when you think it's ready [ Full content available at: https://github.com/apache/dubbo/pull/5071 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org -

[GitHub] [dubbo] htynkn closed pull request #5071: Revert 1 2.7.2 release

2019-09-18 Thread GitHub
[ pull request closed by htynkn ] [ Full content available at: https://github.com/apache/dubbo/pull/5071 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notification

[GitHub] [dubbo] htynkn closed pull request #5070: [Dubbo-4331] Improve api doc #4331

2019-09-18 Thread GitHub
[ pull request closed by htynkn ] [ Full content available at: https://github.com/apache/dubbo/pull/5070 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notification

[GitHub] [dubbo] htynkn commented on issue #4331: [API-DOC] Improve api doc

2019-09-18 Thread GitHub
Script in jenkins job (https://builds.apache.org/job/Apache%20Dubbo/job/apache-dubbo-apidoc-deployment/) also updated [ Full content available at: https://github.com/apache/dubbo/issues/4331 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org --

[GitHub] [dubbo-website] htynkn commented on issue #474: Poor Browser Leverage Caching

2019-09-18 Thread GitHub
because website is hosted in apache server, I'm not sure if it allow custom `.htaccess` for single project site. I'm trying to check if any other apache project website are using `.htaccess` [ Full content available at: https://github.com/apache/dubbo-website/issues/474 ] This message was relay

[GitHub] [dubbo-samples] maslke opened a new issue #118: consumer.xml for dubbo-samples-direct can't work.

2019-09-18 Thread GitBox
maslke opened a new issue #118: consumer.xml for dubbo-samples-direct can't work. URL: https://github.com/apache/dubbo-samples/issues/118 in this demo, there is "url" config in . something like but it does not work. I have to add group and version part to url to make this dem

[GitHub] [dubbo-js] binlaniua opened a new pull request #139: fix network break, heart beat check not work

2019-09-18 Thread GitBox
binlaniua opened a new pull request #139: fix network break, heart beat check not work URL: https://github.com/apache/dubbo-js/pull/139 This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [dubbo-go-hessian2] fangyincheng commented on pull request #131: fix emoji encode error

2019-09-18 Thread GitHub
are `HessianTool` and `HexTool` already used? [ Full content available at: https://github.com/apache/dubbo-go-hessian2/pull/131 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscri