[jira] [Commented] (SCB-771) Update LICENSE/NOTICE for Service-Center 1.0.0 release

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16553762#comment-16553762
 ] 

ASF GitHub Bot commented on SCB-771:


coveralls commented on issue #398: [SCB-771] Add missing dependencies in LICENSE
URL: 
https://github.com/apache/incubator-servicecomb-service-center/pull/398#issuecomment-407270777
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/18128979/badge)](https://coveralls.io/builds/18128979)
   
   Coverage increased (+0.08%) to 74.514% when pulling 
**788c69afdb647c2ebaf022c5c29dc07a13b61c4a on yangbor:master** into 
**aa34cb9d7525a2743b521a8bfe436d0bb4688b12 on apache:master**.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Update LICENSE/NOTICE for Service-Center 1.0.0 release
> --
>
> Key: SCB-771
> URL: https://issues.apache.org/jira/browse/SCB-771
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Service-Center
>Affects Versions: service-center-1.0.0
>Reporter: Yang Bo
>Assignee: Yang Bo
>Priority: Major
> Fix For: service-center-1.0.0
>
>
> Add missing dependencies in LICENSE for binary release.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-771) Update LICENSE/NOTICE for Service-Center 1.0.0 release

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16553761#comment-16553761
 ] 

ASF GitHub Bot commented on SCB-771:


codecov-io commented on issue #398: [SCB-771] Add missing dependencies in 
LICENSE
URL: 
https://github.com/apache/incubator-servicecomb-service-center/pull/398#issuecomment-407270750
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398?src=pr=h1)
 Report
   > Merging 
[#398](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398?src=pr=desc)
 into 
[master](https://codecov.io/gh/apache/incubator-servicecomb-service-center/commit/aa34cb9d7525a2743b521a8bfe436d0bb4688b12?src=pr=desc)
 will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398/graphs/tree.svg?src=pr=GAaF7zrg8R=650=150)](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398?src=pr=tree)
   
   ```diff
   @@   Coverage Diff   @@
   ##   master #398   +/-   ##
   ===
 Coverage   71.69%   71.69%   
   ===
 Files  39   39   
 Lines4130 4130   
   ===
 Hits 2961 2961   
 Misses   1002 1002   
 Partials  167  167
   ```
   
   
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398?src=pr=footer).
 Last update 
[aa34cb9...788c69a](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/398?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Update LICENSE/NOTICE for Service-Center 1.0.0 release
> --
>
> Key: SCB-771
> URL: https://issues.apache.org/jira/browse/SCB-771
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Service-Center
>Affects Versions: service-center-1.0.0
>Reporter: Yang Bo
>Assignee: Yang Bo
>Priority: Major
> Fix For: service-center-1.0.0
>
>
> Add missing dependencies in LICENSE for binary release.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (SCB-772) Java Chassis::Handlers::Loadbalance test failed

2018-07-23 Thread yangyongzheng (JIRA)
yangyongzheng created SCB-772:
-

 Summary: Java Chassis::Handlers::Loadbalance test failed
 Key: SCB-772
 URL: https://issues.apache.org/jira/browse/SCB-772
 Project: Apache ServiceComb
  Issue Type: Bug
  Components: Java-Chassis
Affects Versions: java-chassis-1.0.0-m2
 Environment: JDK : 
java version "1.8.0_171"
Java(TM) SE Runtime Environment (build 1.8.0_171-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.171-b11, mixed mode)

OS : 
Linux version 4.4.0-104-generic (buildd@lgw01-amd64-022) (gcc version 5.4.0 
20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.5) ) #127-Ubuntu SMP Mon Dec 11 12:16:42 
UTC 2017
Reporter: yangyongzheng
 Fix For: java-chassis-1.0.0


Java Chassis::Handlers::Loadbalance test failed: 


2018-07-24 09:45:08,429 [main ] ERROR ExceptionUtils -
Exception: org.apache.servicecomb.core.exception.CseException; No available 
address found. microserviceName=ms, version=null, discoveryGroupName=null
ServiceDefinitionException Code:servicecomb.lb.no.available.address, Message:No 
available address found. microserviceName=ms, version=null, 
discoveryGroupName=null
 at 
org.apache.servicecomb.core.exception.ExceptionUtils.createCseException(ExceptionUtils.java:57)
 at 
org.apache.servicecomb.core.exception.ExceptionUtils.lbAddressNotFound(ExceptionUtils.java:88)
 at 
org.apache.servicecomb.loadbalance.LoadbalanceHandler.send(LoadbalanceHandler.java:135)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:498)
 at mockit.internal.util.MethodReflection.invoke(MethodReflection.java:120)
 at mockit.internal.util.MethodReflection.invoke(MethodReflection.java:185)
 at mockit.Deencapsulation.invoke(Deencapsulation.java:210)
 at 
org.apache.servicecomb.loadbalance.TestLoadbalanceHandler.send_noEndPoint(TestLoadbalanceHandler.java:195)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:498)
 at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
 at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
 at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
 at 
mockit.integration.junit4.internal.JUnit4TestRunnerDecorator.executeTestMethod(JUnit4TestRunnerDecorator.java:162)
 at 
mockit.integration.junit4.internal.JUnit4TestRunnerDecorator.invokeExplosively(JUnit4TestRunnerDecorator.java:71)
 at 
mockit.integration.junit4.internal.MockFrameworkMethod.invokeExplosively(MockFrameworkMethod.java:37)
 at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java)
 at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
 at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
 at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
 at 
org.junit.rules.ExpectedException$ExpectedExceptionStatement.evaluate(ExpectedException.java:239)
 at org.junit.rules.RunRules.evaluate(RunRules.java:20)
 at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
 at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
 at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
 at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
 at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
 at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
 at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
 at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
 at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
 at 
org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:367)
 at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:274)
 at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:238)
 at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:161)
 at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:290)
 at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:242)
 at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:121)

2018-07-24 09:45:08,493 [main ] INFO DiscoveryTree - DiscoveryFilter 
org.apache.servicecomb.loadbalance.filter.ServerDiscoveryFilter, enabled true.
2018-07-24 09:45:08,542 [main ] 

[jira] [Commented] (SCB-771) Update LICENSE/NOTICE for Service-Center 1.0.0 release

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16553752#comment-16553752
 ] 

ASF GitHub Bot commented on SCB-771:


yangbor opened a new pull request #398: [SCB-771] Add missing dependencies in 
LICENSE
URL: https://github.com/apache/incubator-servicecomb-service-center/pull/398
 
 
   Follow this checklist to help us incorporate your contribution quickly and 
easily:
   
- [ ] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/browse/SCB) filed for the change (usually 
before you start working on it).  Trivial changes like typos do not require a 
JIRA issue.  Your pull request should address just this issue, without pulling 
in other changes.
- [ ] Each commit in the pull request should have a meaningful subject line 
and body.
- [ ] Format the pull request title like `[SCB-XXX] Fixes bug in 
ApproximateQuantiles`, where you replace `SCB-XXX` with the appropriate JIRA 
issue.
- [ ] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
- [ ] Run `go build` `go test` `go fmt` `go vet` to make sure basic checks 
pass. A more thorough check will be performed on your pull request 
automatically.
- [ ] If this contribution is large, please file an Apache [Individual 
Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   ---
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Update LICENSE/NOTICE for Service-Center 1.0.0 release
> --
>
> Key: SCB-771
> URL: https://issues.apache.org/jira/browse/SCB-771
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Service-Center
>Affects Versions: service-center-1.0.0
>Reporter: Yang Bo
>Assignee: Yang Bo
>Priority: Major
> Fix For: service-center-1.0.0
>
>
> Add missing dependencies in LICENSE for binary release.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (SCB-771) Update LICENSE/NOTICE for Service-Center 1.0.0 release

2018-07-23 Thread Yang Bo (JIRA)
Yang Bo created SCB-771:
---

 Summary: Update LICENSE/NOTICE for Service-Center 1.0.0 release
 Key: SCB-771
 URL: https://issues.apache.org/jira/browse/SCB-771
 Project: Apache ServiceComb
  Issue Type: Improvement
  Components: Service-Center
Affects Versions: service-center-1.0.0
Reporter: Yang Bo
Assignee: Yang Bo
 Fix For: service-center-1.0.0


Add missing dependencies in LICENSE for binary release.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-687) highway server not accept too many connection

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-687?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16553715#comment-16553715
 ] 

ASF GitHub Bot commented on SCB-687:


zhengyangyong commented on a change in pull request #824: [SCB-687] add highway 
server connection protection
URL: 
https://github.com/apache/incubator-servicecomb-java-chassis/pull/824#discussion_r204609279
 
 

 ##
 File path: 
integration-tests/spring-pojo-connection-limit-test/src/test/resources/microservice.yaml
 ##
 @@ -0,0 +1,29 @@
+## ---
+## Licensed to the Apache Software Foundation (ASF) under one or more
+## contributor license agreements.  See the NOTICE file distributed with
+## this work for additional information regarding copyright ownership.
+## The ASF licenses this file to You under the Apache License, Version 2.0
+## (the "License"); you may not use this file except in compliance with
+## the License.  You may obtain a copy of the License at
+##
+##  http://www.apache.org/licenses/LICENSE-2.0
+##
+## Unless required by applicable law or agreed to in writing, software
+## distributed under the License is distributed on an "AS IS" BASIS,
+## WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+## See the License for the specific language governing permissions and
+## limitations under the License.
+## ---
+
+APPLICATION_ID: pojotest-it
+service_description:
+  name: pojo
+  version: 0.0.4
+servicecomb:
+  service:
+registry:
+  address: http://127.0.0.1:30100
+  highway:
+address: 0.0.0.0:7070
+server:
+  connection-limit: 0
 
 Review comment:
   Done


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> highway server not accept too many connection
> -
>
> Key: SCB-687
> URL: https://issues.apache.org/jira/browse/SCB-687
> Project: Apache ServiceComb
>  Issue Type: Sub-task
>  Components: Java-Chassis
>Reporter: wujimin
>Assignee: yangyongzheng
>Priority: Major
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-735) Add admin dump api

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16553647#comment-16553647
 ] 

ASF GitHub Bot commented on SCB-735:


coveralls edited a comment on issue #396: SCB-735 Add admin dump api
URL: 
https://github.com/apache/incubator-servicecomb-service-center/pull/396#issuecomment-406771280
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/18127937/badge)](https://coveralls.io/builds/18127937)
   
   Coverage increased (+1.6%) to 76.049% when pulling 
**7242ebf83e096b1cbb5cb65dbbefe9397e33e73e on little-cui:admin** into 
**aa34cb9d7525a2743b521a8bfe436d0bb4688b12 on apache:master**.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Add admin dump api
> --
>
> Key: SCB-735
> URL: https://issues.apache.org/jira/browse/SCB-735
> Project: Apache ServiceComb
>  Issue Type: New Feature
>  Components: Service-Center
>Reporter: little-cui
>Assignee: little-cui
>Priority: Major
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-735) Add admin dump api

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16553645#comment-16553645
 ] 

ASF GitHub Bot commented on SCB-735:


codecov-io edited a comment on issue #396: SCB-735 Add admin dump api
URL: 
https://github.com/apache/incubator-servicecomb-service-center/pull/396#issuecomment-406771272
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=h1)
 Report
   > Merging 
[#396](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=desc)
 into 
[master](https://codecov.io/gh/apache/incubator-servicecomb-service-center/commit/aa34cb9d7525a2743b521a8bfe436d0bb4688b12?src=pr=desc)
 will **increase** coverage by `1.1%`.
   > The diff coverage is `77.25%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/graphs/tree.svg?height=150=650=GAaF7zrg8R=pr)](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=tree)
   
   ```diff
   @@   Coverage Diff@@
   ##   master#396 +/-   ##
   
   + Coverage   71.69%   72.8%   +1.1% 
   
 Files  39 118 +79 
 Lines41309743   +5613 
   
   + Hits 29617093   +4132 
   - Misses   10022174   +1172 
   - Partials  167 476+309
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=tree)
 | Coverage Δ | |
   |---|---|---|
   | 
[pkg/chain/invocation.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL2NoYWluL2ludm9jYXRpb24uZ28=)
 | `89.47% <ø> (ø)` | |
   | 
[pkg/tlsutil/tlsutil.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3Rsc3V0aWwvdGxzdXRpbC5nbw==)
 | `73.07% <ø> (ø)` | |
   | 
[pkg/task/executor.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3Rhc2svZXhlY3V0b3IuZ28=)
 | `76% <ø> (ø)` | |
   | 
[server/service/notification/common.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL3NlcnZpY2Uvbm90aWZpY2F0aW9uL2NvbW1vbi5nbw==)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[server/core/backend/extend.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2NvcmUvYmFja2VuZC9leHRlbmQuZ28=)
 | `75% <ø> (ø)` | |
   | 
[server/error/error.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2Vycm9yL2Vycm9yLmdv)
 | `100% <ø> (ø)` | |
   | 
[server/govern/controller\_v4.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2dvdmVybi9jb250cm9sbGVyX3Y0Lmdv)
 | `73.49% <ø> (ø)` | |
   | 
[pkg/validate/url.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3ZhbGlkYXRlL3VybC5nbw==)
 | `0% <0%> (ø)` | |
   | 
[server/core/backend/common.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2NvcmUvYmFja2VuZC9jb21tb24uZ28=)
 | `75% <0%> (ø)` | |
   | 
[pkg/task/service.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3Rhc2svc2VydmljZS5nbw==)
 | `40% <0%> (ø)` | |
   | ... and [127 
more](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=footer).
 Last update 
[aa34cb9...7242ebf](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Add admin dump api
> --
>
> Key: SCB-735
> URL: https://issues.apache.org/jira/browse/SCB-735
> Project: Apache ServiceComb
>  Issue Type: New Feature
>  Components: 

[jira] [Resolved] (SCB-761) Prepare 1.0.0 Release for SC

2018-07-23 Thread Mohammad Asif Siddiqui (JIRA)


 [ 
https://issues.apache.org/jira/browse/SCB-761?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mohammad Asif Siddiqui resolved SCB-761.

Resolution: Fixed

> Prepare 1.0.0 Release for SC
> 
>
> Key: SCB-761
> URL: https://issues.apache.org/jira/browse/SCB-761
> Project: Apache ServiceComb
>  Issue Type: Task
>  Components: Service-Center
>Reporter: Mohammad Asif Siddiqui
>Assignee: Mohammad Asif Siddiqui
>Priority: Major
> Fix For: service-center-1.0.0
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-767) When using Response to return multi-code message, handlers / filters given ClassCaseException

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552819#comment-16552819
 ] 

ASF GitHub Bot commented on SCB-767:


coveralls edited a comment on issue #827: [SCB-767]When using Response to 
return multi-code message, handlers /filters given ClassCaseException
URL: 
https://github.com/apache/incubator-servicecomb-java-chassis/pull/827#issuecomment-406578556
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/18115081/badge)](https://coveralls.io/builds/18115081)
   
   Coverage decreased (-0.3%) to 86.668% when pulling 
**a2722a7d74c1fc91c3922fe0c4d7c49cb7788b82 on liubao68:response-fail** into 
**a1d0804132cc949061440d7de6fd971a1cfeb246 on apache:master**.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> When using Response to return multi-code message, handlers / filters given 
> ClassCaseException
> -
>
> Key: SCB-767
> URL: https://issues.apache.org/jira/browse/SCB-767
> Project: Apache ServiceComb
>  Issue Type: Bug
>Reporter: liubao
>Assignee: liubao
>Priority: Major
>
> Error code 400, 500 etc. takes as error. But the object result is not 
> throwable. So got class cast exception. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-735) Add admin dump api

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552816#comment-16552816
 ] 

ASF GitHub Bot commented on SCB-735:


codecov-io edited a comment on issue #396: SCB-735 Add admin dump api
URL: 
https://github.com/apache/incubator-servicecomb-service-center/pull/396#issuecomment-406771272
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=h1)
 Report
   > Merging 
[#396](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=desc)
 into 
[master](https://codecov.io/gh/apache/incubator-servicecomb-service-center/commit/aa34cb9d7525a2743b521a8bfe436d0bb4688b12?src=pr=desc)
 will **increase** coverage by `1.1%`.
   > The diff coverage is `77.02%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/graphs/tree.svg?width=650=150=GAaF7zrg8R=pr)](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=tree)
   
   ```diff
   @@   Coverage Diff@@
   ##   master#396 +/-   ##
   
   + Coverage   71.69%   72.8%   +1.1% 
   
 Files  39 118 +79 
 Lines41309743   +5613 
   
   + Hits 29617093   +4132 
   - Misses   10022174   +1172 
   - Partials  167 476+309
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=tree)
 | Coverage Δ | |
   |---|---|---|
   | 
[pkg/tlsutil/tlsutil.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3Rsc3V0aWwvdGxzdXRpbC5nbw==)
 | `74.03% <ø> (ø)` | |
   | 
[server/core/backend/extend.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2NvcmUvYmFja2VuZC9leHRlbmQuZ28=)
 | `75% <ø> (ø)` | |
   | 
[pkg/chain/invocation.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL2NoYWluL2ludm9jYXRpb24uZ28=)
 | `89.47% <ø> (ø)` | |
   | 
[pkg/task/executor.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3Rhc2svZXhlY3V0b3IuZ28=)
 | `76% <ø> (ø)` | |
   | 
[server/error/error.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2Vycm9yL2Vycm9yLmdv)
 | `100% <ø> (ø)` | |
   | 
[server/service/notification/common.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL3NlcnZpY2Uvbm90aWZpY2F0aW9uL2NvbW1vbi5nbw==)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[server/govern/controller\_v4.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2dvdmVybi9jb250cm9sbGVyX3Y0Lmdv)
 | `73.49% <ø> (ø)` | |
   | 
[pkg/validate/url.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3ZhbGlkYXRlL3VybC5nbw==)
 | `0% <0%> (ø)` | |
   | 
[server/core/backend/common.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-c2VydmVyL2NvcmUvYmFja2VuZC9jb21tb24uZ28=)
 | `75% <0%> (ø)` | |
   | 
[pkg/task/service.go](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree#diff-cGtnL3Rhc2svc2VydmljZS5nbw==)
 | `40% <0%> (ø)` | |
   | ... and [127 
more](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=footer).
 Last update 
[aa34cb9...9c673ac](https://codecov.io/gh/apache/incubator-servicecomb-service-center/pull/396?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Add admin dump api
> --
>
> Key: SCB-735
> URL: https://issues.apache.org/jira/browse/SCB-735
> Project: Apache ServiceComb
>  Issue Type: New Feature
>  Components: 

[jira] [Commented] (SCB-735) Add admin dump api

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552815#comment-16552815
 ] 

ASF GitHub Bot commented on SCB-735:


coveralls edited a comment on issue #396: SCB-735 Add admin dump api
URL: 
https://github.com/apache/incubator-servicecomb-service-center/pull/396#issuecomment-406771280
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/18115063/badge)](https://coveralls.io/builds/18115063)
   
   Coverage increased (+1.6%) to 76.063% when pulling 
**9c673ac19c3007bb6839379c60e7dbaa161166b1 on little-cui:admin** into 
**aa34cb9d7525a2743b521a8bfe436d0bb4688b12 on apache:master**.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Add admin dump api
> --
>
> Key: SCB-735
> URL: https://issues.apache.org/jira/browse/SCB-735
> Project: Apache ServiceComb
>  Issue Type: New Feature
>  Components: Service-Center
>Reporter: little-cui
>Assignee: little-cui
>Priority: Major
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-767) When using Response to return multi-code message, handlers / filters given ClassCaseException

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552773#comment-16552773
 ] 

ASF GitHub Bot commented on SCB-767:


liubao68 commented on issue #827: [SCB-767]When using Response to return 
multi-code message, handlers /filters given ClassCaseException
URL: 
https://github.com/apache/incubator-servicecomb-java-chassis/pull/827#issuecomment-407044680
 
 
   After modification to service implementation, this scenario is not a bug. 
But user may have the same problem when they use it this way, just commit the 
test codes as an example. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> When using Response to return multi-code message, handlers / filters given 
> ClassCaseException
> -
>
> Key: SCB-767
> URL: https://issues.apache.org/jira/browse/SCB-767
> Project: Apache ServiceComb
>  Issue Type: Bug
>Reporter: liubao
>Assignee: liubao
>Priority: Major
>
> Error code 400, 500 etc. takes as error. But the object result is not 
> throwable. So got class cast exception. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-736) generate default value to swagger for primitive type, even there is no defaultValue annotation

2018-07-23 Thread Mahesh Raju Somalaraju (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552634#comment-16552634
 ] 

Mahesh Raju Somalaraju commented on SCB-736:


This issue is in progress

> generate default value to swagger for primitive type, even there is no 
> defaultValue annotation
> --
>
> Key: SCB-736
> URL: https://issues.apache.org/jira/browse/SCB-736
> Project: Apache ServiceComb
>  Issue Type: Task
>  Components: Java-Chassis
>Reporter: wujimin
>Assignee: Mahesh Raju Somalaraju
>Priority: Major
>
> {code:java}
> @GetMapping(path = "test")
> public String test(int i,  boolean b) {
>   return "" + i + b;
> }
> {code}
> request without any query parameter should be ok
> must include all primitive types.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Assigned] (SCB-736) generate default value to swagger for primitive type, even there is no defaultValue annotation

2018-07-23 Thread Mahesh Raju Somalaraju (JIRA)


 [ 
https://issues.apache.org/jira/browse/SCB-736?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahesh Raju Somalaraju reassigned SCB-736:
--

Assignee: Mahesh Raju Somalaraju  (was: sukesh)

> generate default value to swagger for primitive type, even there is no 
> defaultValue annotation
> --
>
> Key: SCB-736
> URL: https://issues.apache.org/jira/browse/SCB-736
> Project: Apache ServiceComb
>  Issue Type: Task
>  Components: Java-Chassis
>Reporter: wujimin
>Assignee: Mahesh Raju Somalaraju
>Priority: Major
>
> {code:java}
> @GetMapping(path = "test")
> public String test(int i,  boolean b) {
>   return "" + i + b;
> }
> {code}
> request without any query parameter should be ok
> must include all primitive types.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-769) When delay fault injection is enabled, the business thread will be blocked

2018-07-23 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552563#comment-16552563
 ] 

ASF GitHub Bot commented on SCB-769:


yhs0092 opened a new pull request #833: [SCB-769] Fix delay fault injection 
blocking thread
URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/833
 
 
   Follow this checklist to help us incorporate your contribution quickly and 
easily:
   
- [x] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/browse/SCB) filed for the change (usually 
before you start working on it).  Trivial changes like typos do not require a 
JIRA issue.  Your pull request should address just this issue, without pulling 
in other changes.
- [x] Each commit in the pull request should have a meaningful subject line 
and body.
- [x] Format the pull request title like `[SCB-XXX] Fixes bug in 
ApproximateQuantiles`, where you replace `SCB-XXX` with the appropriate JIRA 
issue.
- [x] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
- [x] Run `mvn clean install` to make sure basic checks pass. A more 
thorough check will be performed on your pull request automatically.
- [ ] If this contribution is large, please file an Apache [Individual 
Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   ---
   See details in [SCB-769](https://issues.apache.org/jira/browse/SCB-769)


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> When delay fault injection is enabled, the business thread will be blocked 
> ---
>
> Key: SCB-769
> URL: https://issues.apache.org/jira/browse/SCB-769
> Project: Apache ServiceComb
>  Issue Type: Bug
>Reporter: YaoHaishi
>Assignee: YaoHaishi
>Priority: Major
> Fix For: java-chassis-1.1.0
>
>
> When delay fault injection is enabled, the not delayed invocation will make 
> business thread blocked in synchronized mode, or network thread blocked in 
> reactive mode.
> The cause is that if the invocation should not be delayed, the process logic 
> in DelayFault exits without calling asynResponse.success("success");



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-709) Implement JAX-RS body parameter required feature

2018-07-23 Thread Mahesh Raju Somalaraju (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-709?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16552510#comment-16552510
 ] 

Mahesh Raju Somalaraju commented on SCB-709:


1. Implement JAX-RS body parameter required feature

>> Regarding this implentation, Jaxrs not support for body as optional.  

      "body parameter required false" option is not there.

 

Closing this Jira as point 1 and point 2 has nothing to do it. 

> Implement JAX-RS body parameter required feature
> 
>
> Key: SCB-709
> URL: https://issues.apache.org/jira/browse/SCB-709
> Project: Apache ServiceComb
>  Issue Type: New Feature
>Reporter: liubao
>Assignee: Mahesh Raju Somalaraju
>Priority: Major
>
> SCB-701 remaining problems:
> 1. Implement JAX-RS body parameter required feature
> 2. SCB-701 has changes, consumer code can send null or an object to provider, 
> null encode as empty body I think is better. And for provider, is required = 
> true, either null or emty body will throw exception is better, but now from 
> consumer code, we can not send null. and with postman, send null will not 
> throw exception when required = true
> 3. 
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)