[dubbo-samples] branch master updated: move nacos tests into another profile

2019-08-13 Thread iluo
This is an automated email from the ASF dual-hosted git repository. iluo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/dubbo-samples.git The following commit(s) were added to refs/heads/master by this push: new bfd820f move nacos tests into another prof

[dubbo-samples] branch master updated: not sure why nacos integration tests are unstable on travis, disable for now

2019-08-13 Thread iluo
This is an automated email from the ASF dual-hosted git repository. iluo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/dubbo-samples.git The following commit(s) were added to refs/heads/master by this push: new a2b6a0a not sure why nacos integration tes

[GitHub] [dubbo] tswstarplanet commented on issue #4772: spirng context中存在匿名bean时,dubbo初始化抛npe

2019-08-13 Thread GitHub
@lixiaojie 这个类的beanName属性可能为null吗? [ Full content available at: https://github.com/apache/dubbo/issues/4772 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificati

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #116: decode failed when invoke to java server ,and return specially

2019-08-13 Thread GitHub
@pantianying have you try HashMap instead of JSONObject, try ArrayList instead of JSONArray? [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/116 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org ---

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #115: 序列化数组时报no this type name

2019-08-13 Thread GitHub
@zhou27 export the struct and its fields [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/115 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe,

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #104: StringBuilder & StringBuffer 使用string接收会报错

2019-08-13 Thread GitHub
@pepigo no use special Object only exists in java, just using String instead. [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/104 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #104: StringBuilder & StringBuffer 使用string接收会报错

2019-08-13 Thread GitHub
@pepigo not use special Object only exists in java, just using String instead. [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/104 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org ---

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #100: Bug: 各种包装类型零值问题

2019-08-13 Thread GitHub
@pepigo I don't think it's a good idea to add more `**basic data type**` mapping , which is out of the scope of hessian serialization protocol. Another data type like BigDecimal, which is used widely in business and no solution for it, then we consider to support it. see https://github.com/h

[GitHub] [dubbo-go-hessian2] wongoo closed issue #90: can't decode null java date

2019-08-13 Thread GitHub
[ issue closed by wongoo ] [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/90 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifi

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #52: Ftr: We should allow the passed []struct type in the argument instead of []interface{}

2019-08-13 Thread GitHub
@hxmhlt have u tried it again? [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/52 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: no

[GitHub] [dubbo-go-hessian2] wongoo commented on issue #51: Imp: hessian2 should handle all errors of encode and decode

2019-08-13 Thread GitHub
@u0x01 if there is not problem, we will close it. @AlexStocks [ Full content available at: https://github.com/apache/dubbo-go-hessian2/issues/51 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [dubbo] cvictory closed pull request #4795: issue #4238: provider服务端配置serialization="kryo"自定义异常类抛错时消费端捕获不到

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

[GitHub] [dubbo] cvictory closed issue #4238: provider服务端配置serialization="kryo"自定义异常类抛错时消费端捕获不到

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

[GitHub] [dubbo] mercyblitz opened issue #4815: [Feature] Add the ServiceLoader for Dubbo's services or components

2019-08-13 Thread GitHub
An extension class of ServiceLoader to help the ease for use of Dubbo services/components [ Full content available at: https://github.com/apache/dubbo/issues/4815 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org ---

[GitHub] [dubbo] FelixFly commented on issue #4742: @Method retries set not work

2019-08-13 Thread GitHub
> Should be set this way > > Because `ServiceConfig` and `ReferenceConfig` will respectively intercept the > `@Service `and `@Reference` annotations, and finally add the attributes in > the annotation to the registration url. > > @reference(version = "1.0.0", timeout = 1, retries=-1) This

[GitHub] [dubbo-go] sxllwx commented on issue #163: Memory leak in protocol dubbo maybe

2019-08-13 Thread GitHub
>From pprof file, I got >result.![image](https://user-images.githubusercontent.com/11829907/62992075-ad105500-be84-11e9-8f71-dd22b4e8aa02.png) Lot of memory used by github.com/apache/dubbo-go/protocol/dubbo.NewPendingResponse function then store the object to sync.Map , but not clean after hea

[GitHub] [dubbo] tswstarplanet commented on issue #4816: Is it appropriate about the suffix of ServiceAnnotationBeanPostProcessor class name?

2019-08-13 Thread GitHub
I think you are right. [ Full content available at: https://github.com/apache/dubbo/issues/4816 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubsc

[GitHub] [dubbo] mercyblitz opened pull request #4817: Dubbo cloud native

2019-08-13 Thread GitHub
[ Full content available at: https://github.com/apache/dubbo/pull/4817 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org Fo

[GitHub] [dubbo-go] AlexStocks commented on issue #166: [ISSUE #163] clean heartbeat from PendingResponse

2019-08-13 Thread GitHub
LGTM [ Full content available at: https://github.com/apache/dubbo-go/pull/166 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.

[GitHub] [dubbo] codecov-io commented on issue #4776: [Dubbo-4771]fix protocol register set false not work

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

[GitHub] [dubbo-samples] complone commented on issue #101: dubbo-samples-generic-impl-provider cannot pass travis CI

2019-08-13 Thread GitBox
complone commented on issue #101: dubbo-samples-generic-impl-provider cannot pass travis CI URL: https://github.com/apache/dubbo-samples/issues/101#issuecomment-521091222 I think you should configure a docker environment on the remote host and debug it on it, because the successfully packa

[GitHub] [dubbo-go] AlexStocks closed pull request #166: [ISSUE #163] clean heartbeat from PendingResponse

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

[GitHub] [dubbo-website] purple-force commented on issue #446: The whole ecology page of Dubbo

2019-08-13 Thread GitHub
rm -rf node_modules && npm i [ Full content available at: https://github.com/apache/dubbo-website/pull/446 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notificati

[GitHub] [dubbo-website] lovepoem commented on issue #446: The whole ecology page of Dubbo

2019-08-13 Thread GitHub
> rm -rf node_modules && npm i I did it , now have a new error ```perilog wxdeMacBook-Pro:dubbo-website wx$ docsite build assert.js:42 throw new errors.AssertionError({ ^ AssertionError [ERR_ASSERTION]: Task function must be specified at Gulp.set [as _setTask] (/Users/wx/projects/dubbo-

[GitHub] [dubbo-website] lovepoem commented on issue #446: The whole ecology page of Dubbo

2019-08-13 Thread GitHub
> rm -rf node_modules && npm i I did it , now have a new error ```verilog wxdeMacBook-Pro:dubbo-website wx$ docsite build assert.js:42 throw new errors.AssertionError({ ^ AssertionError [ERR_ASSERTION]: Task function must be specified at Gulp.set [as _setTask] (/Users/wx/projects/dubbo-

[GitHub] [dubbo] tswstarplanet commented on issue #4775: get wrong default protocolConfig from another service.

2019-08-13 Thread GitHub
I check the code, it's because when you export the DemoService2, the protocol is stored in ConfigManager as a "default" protocol, and when the DemoService is exported, the "default" protocol will be set to it. [ Full content available at: https://github.com/apache/dubbo/issues/4775 ] This messa

[GitHub] [dubbo-js] xusd320 opened a new issue #130: zookeep的acl认证支持

2019-08-13 Thread GitBox
xusd320 opened a new issue #130: zookeep的acl认证支持 URL: https://github.com/apache/dubbo-js/issues/130 node-zookeeper-client本身是支持zk连接配置权限认证的,详见:https://www.npmjs.com/package/node-zookeeper-client#void-addauthinfoscheme-auth dubbo2.js是否能添加相关支持, 在register中传入配置?如下: { url: 'localhost:31

[GitHub] [dubbo-js] xusd320 opened issue #130: zookeep的acl认证支持

2019-08-13 Thread GitHub
node-zookeeper-client本身是支持zk连接配置权限认证的,详见:https://www.npmjs.com/package/node-zookeeper-client#void-addauthinfoscheme-auth dubbo2.js是否能添加相关支持, 在register中传入配置?如下: { url: 'localhost:3128', authInfo: { scheme: 'digest', auth: 'user:123456' } } 在调用zookooper.createClient之后调用zookeeper.addAuthInfo,即可实现

[GitHub] [dubbo] wavesZh commented on issue #4732: Dubbo 2.6.6 客户端调用timeout 设置在特殊情况下未起作用

2019-08-13 Thread GitHub
Your case occur frequently?Maybe STW? #40. [ Full content available at: https://github.com/apache/dubbo/issues/4732 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail:

[GitHub] [dubbo] z529192557 commented on issue #4784: Use generic invocation via API , lost "version" value

2019-08-13 Thread GitHub
> test Yes,it‘s direct invocation,only in direct invocation via API, can reproduce. [ Full content available at: https://github.com/apache/dubbo/issues/4784 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org --

[GitHub] [dubbo]

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

[GitHub] [dubbo] z529192557 commented on issue #4784: Use generic invocation via API , lost "version" value

2019-08-13 Thread GitHub
> > @z529192557 but I tried it on main branch, but failed to reproduce. > > I didn't realize your test is direct invocation, pls. ignore my previous > comment. Yes,it‘s direct invocation,only in direct invocation via API, can reproduce. [ Full content available at: https://github.com/apache/dub

[GitHub] [dubbo] haiyang1985 commented on issue #4775: get wrong default protocolConfig from another service.

2019-08-13 Thread GitHub
@tswstarplanet yes. [ Full content available at: https://github.com/apache/dubbo/issues/4775 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscribe, e-mail: notifications-unsubscr.

[GitHub] [dubbo] beiwei30 opened pull request #4818: enhance consul registry to support matching url between consumer and provider

2019-08-13 Thread GitHub
## What is the purpose of the change enhance consul registry to support matching url between consumer and provider ## Brief changelog X ## Verifying this change X Follow this checklist to help us incorporate your contribution quickly and easily: - [x] Make sure there is a [GITHUB