[GitHub] wujimin commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
wujimin commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636#discussion_r179938700 ## File path:

[GitHub] coveralls commented on issue #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
coveralls commented on issue #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636#issuecomment-379521919 [![Coverage

[GitHub] liubao68 closed pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
liubao68 closed pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] liubao68 commented on a change in pull request #635: [SCB-462] cloud eye publisher switch to new mechanism

2018-04-08 Thread GitBox
liubao68 commented on a change in pull request #635: [SCB-462] cloud eye publisher switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/635#discussion_r179944492 ## File path:

[GitHub] liubao68 commented on a change in pull request #635: [SCB-462] cloud eye publisher switch to new mechanism

2018-04-08 Thread GitBox
liubao68 commented on a change in pull request #635: [SCB-462] cloud eye publisher switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/635#discussion_r179944492 ## File path:

Re: [VOTE] Holding Apache ServiceComb (incubating) Meetup Hosted by Huawei Cloud as a Co-located Event in LC3 2018 Aisa

2018-04-08 Thread Willem Jiang
+1 (Binding) Willem Jiang Blog: http://willemjiang.blogspot.com (English) http://jnn.iteye.com (Chinese) Twitter: willemjiang Weibo: 姜宁willem On Sun, Apr 8, 2018 at 3:40 PM, Zen Lin wrote: > Hi All, > This is a call for Vote to hold Apache ServiceComb

[GitHub] wujimin commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
wujimin commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636#discussion_r179938611 ## File path:

[VOTE] Holding Apache ServiceComb (incubating) Meetup Hosted by Huawei Cloud as a Co-located Event in LC3 2018 Aisa

2018-04-08 Thread Zen Lin
Hi All, This is a call for Vote to hold Apache ServiceComb (incubating) Meetup Hosted by Huawei Cloud as a Co-located Event in LC3 2018 Aisa. As discussed in the meetup propersal[1], we have plan to hold Apache ServiceComb (incubating) Mini Meetup, details are listed bellow, - What is the

[GitHub] lijasonvip commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
lijasonvip commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636#discussion_r179939355 ## File path:

[GitHub] coveralls commented on issue #635: [SCB-462] cloud eye publisher switch to new mechanism

2018-04-08 Thread GitBox
coveralls commented on issue #635: [SCB-462] cloud eye publisher switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/635#issuecomment-379478939 [![Coverage

[GitHub] wujimin opened a new pull request #639: [SCB-422] prometheus switch to new mechanism

2018-04-08 Thread GitBox
wujimin opened a new pull request #639: [SCB-422] prometheus switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/639 ![image](https://user-images.githubusercontent.com/16874843/38469195-56cbd4b0-3b83-11e8-90d9-8ed1cc627144.png)

[GitHub] lijasonvip commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
lijasonvip commented on a change in pull request #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636#discussion_r179939180 ## File path:

[GitHub] liubao68 closed pull request #630: [SCB-444]try to optimize autoDiscovery function

2018-04-08 Thread GitBox
liubao68 closed pull request #630: [SCB-444]try to optimize autoDiscovery function URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/630 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] liubao68 closed pull request #629: fix typo change swagger to hystrix

2018-04-08 Thread GitBox
liubao68 closed pull request #629: fix typo change swagger to hystrix URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/629 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] liubao68 commented on issue #629: fix typo change swagger to hystrix

2018-04-08 Thread GitBox
liubao68 commented on issue #629: fix typo change swagger to hystrix URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/629#issuecomment-379535277 Close as merged in https://github.com/apache/incubator-servicecomb-java-chassis/pull/638

[GitHub] liubao68 opened a new issue #324: sc front end service name is not correct

2018-04-08 Thread GitBox
liubao68 opened a new issue #324: sc front end service name is not correct URL: https://github.com/apache/incubator-servicecomb-service-center/issues/324 service name is case sensitive, but sc front end will rename the first letter to upper case. this is quit confusing.

[GitHub] liubao68 commented on a change in pull request #633: [SCB-422] Threadpool metrics

2018-04-08 Thread GitBox
liubao68 commented on a change in pull request #633: [SCB-422] Threadpool metrics URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/633#discussion_r179940415 ## File path:

[GitHub] wujimin closed pull request #633: [SCB-422] Threadpool metrics

2018-04-08 Thread GitBox
wujimin closed pull request #633: [SCB-422] Threadpool metrics URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/633 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] wujimin commented on a change in pull request #635: [SCB-462] cloud eye publisher switch to new mechanism

2018-04-08 Thread GitBox
wujimin commented on a change in pull request #635: [SCB-462] cloud eye publisher switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/635#discussion_r179950917 ## File path:

[GitHub] liubao68 closed pull request #638: [SCB-449] Fix typo in LICENSE files

2018-04-08 Thread GitBox
liubao68 closed pull request #638: [SCB-449] Fix typo in LICENSE files URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/638 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

Re: [Discussion]Provider a project fully show all features of ServiceComb(also can use as a scaffold)

2018-04-08 Thread Willem Jiang
I just went through the documents, it looks good. Here are some comments for the edge service part. For the canary test, we need to make sure the new version of service is available to some kind of user, not the normal round robin one. Is better to show the user to do AB test by applying different

[GitHub] lijasonvip commented on issue #636: SCB-189 fix CI problems and warnings for feature support async restTe…

2018-04-08 Thread GitBox
lijasonvip commented on issue #636: SCB-189 fix CI problems and warnings for feature support async restTe… URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/636#issuecomment-379526718 ![no

[GitHub] yangbor opened a new pull request #638: [SCB-449] Fix typo in LICENSE files

2018-04-08 Thread GitBox
yangbor opened a new pull request #638: [SCB-449] Fix typo in LICENSE files URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/638 Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] Make sure there is a [JIRA

[GitHub] wujimin commented on a change in pull request #633: [SCB-422] Threadpool metrics

2018-04-08 Thread GitBox
wujimin commented on a change in pull request #633: [SCB-422] Threadpool metrics URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/633#discussion_r179941587 ## File path:

Re: Request the permissions of using name and logo of ServiceComb in the Meetup

2018-04-08 Thread Kevin A. McGrail
Happy to help. On Sun, Apr 8, 2018, 02:50 Zen Lin wrote: > Hi Sally, Kevin, > > I am going to have a fromal vote email to get approval of PMCs, and then to > get permissions from VP and Brand Management. > > Thanks for your help. > > 2018-04-04 18:17 GMT+08:00 Sally

[GitHub] coveralls commented on issue #639: [SCB-422] prometheus switch to new mechanism

2018-04-08 Thread GitBox
coveralls commented on issue #639: [SCB-422] prometheus switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/639#issuecomment-379563648 [![Coverage Status](https://coveralls.io/builds/16406707/badge)](https://coveralls.io/builds/16406707)

[GitHub] wujimin closed pull request #635: [SCB-462] cloud eye publisher switch to new mechanism

2018-04-08 Thread GitBox
wujimin closed pull request #635: [SCB-462] cloud eye publisher switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/635 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] liubao68 closed pull request #637: [SCB-459]Update netty's version to 4.1.19 and vertx's version to 3.5.1

2018-04-08 Thread GitBox
liubao68 closed pull request #637: [SCB-459]Update netty's version to 4.1.19 and vertx's version to 3.5.1 URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/637 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

?????? [Discussion]Provider a project fully show all features ofServiceComb(also can use as a scaffold)

2018-04-08 Thread bismy
This is a good idea, and the features is good too. Here are some works done that you can refer to https://java.huaweicse.com/featured-topics/develope-microservice-using-ces.html including microservice design and the working code is also included. If you want to demonstrate transactions

Re: [Discussion]Provider a project fully show all features ofServiceComb(also can use as a scaffold)

2018-04-08 Thread wjm wjm
we already have some projects like this(houseApp or something else). so my question is: design some new projects or make the exists better. 2018-04-09 11:02 GMT+08:00 bismy : > This is a good idea, and the features is good too. > Here are some works done that you can refer to >

?????? [VOTE] Holding Apache ServiceComb (incubating) Meetup Hosted byHuawei Cloud as a Co-located Event in LC3 2018 Aisa

2018-04-08 Thread bismy
+1 ??Binding?? -- -- ??: "willem.jiang"; : 2018??4??8??(??) 10:13 ??: "dev"; : Re: [VOTE] Holding Apache ServiceComb (incubating) Meetup Hosted byHuawei Cloud as a

[GitHub] liubao68 commented on a change in pull request #639: [SCB-422] prometheus switch to new mechanism

2018-04-08 Thread GitBox
liubao68 commented on a change in pull request #639: [SCB-422] prometheus switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/639#discussion_r179981976 ## File path:

[GitHub] liubao68 commented on a change in pull request #639: [SCB-422] prometheus switch to new mechanism

2018-04-08 Thread GitBox
liubao68 commented on a change in pull request #639: [SCB-422] prometheus switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/639#discussion_r179981976 ## File path:

[GitHub] liubao68 commented on a change in pull request #632: [SCB-456]Provider a way to input configuration from a Map

2018-04-08 Thread GitBox
liubao68 commented on a change in pull request #632: [SCB-456]Provider a way to input configuration from a Map URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/632#discussion_r179982498 ## File path:

[GitHub] acsukesh commented on a change in pull request #627: [SCB-292] chassis support standard parameter validation

2018-04-08 Thread GitBox
acsukesh commented on a change in pull request #627: [SCB-292] chassis support standard parameter validation URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/627#discussion_r179983363 ## File path:

Re: [Discussion]Provider a project fully show all features of ServiceComb(also can use as a scaffold)

2018-04-08 Thread wjm wjm
AB test supported by DarkLaunchFilter in CSE not ServiceComb 2018-04-08 22:29 GMT+08:00 Willem Jiang : > I just went through the documents, it looks good. > Here are some comments for the edge service part. > For the canary test, we need to make sure the new version of

[GitHub] asifdxtreme opened a new pull request #59: Add the Release guide for ServiceComb

2018-04-08 Thread GitBox
asifdxtreme opened a new pull request #59: Add the Release guide for ServiceComb URL: https://github.com/apache/incubator-servicecomb-website/pull/59 Preview : https://asifdxtreme.github.io/incubator-servicecomb-website/developers/release-guide/

[GitHub] wujimin commented on a change in pull request #639: [SCB-422] prometheus switch to new mechanism

2018-04-08 Thread GitBox
wujimin commented on a change in pull request #639: [SCB-422] prometheus switch to new mechanism URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/639#discussion_r179985745 ## File path: