adjust error handling strategy - Githubissues

thubbo / jmeter-plugins-for-apache-dubbo

Dubbo Plugin for Apache JMeter, It is a plug-in developed for testing Dubbo in Jmeter.

Apache License 2.0

566 stars 289 forks source link

adjust error handling strategy #95

Closed sofkyle closed 5 years ago

sofkyle commented 5 years ago

What is the purpose of the change

1、adjust error handling strategy #94 2、optimize some code and method name

Brief changelog

src/main/java/io/github/ningyu/jmeter/plugin/dubbo/sample/DubboSample.java src/main/java/io/github/ningyu/jmeter/plugin/dubbo/gui/DubboCommonPanel.java src/main/java/io/github/ningyu/jmeter/plugin/dubbo/sample/ProviderService.java

Verifying this change

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[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 -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.

ningyu1 commented 5 years ago

@sofkyle 分支应该into 到2.7.x中，非master

ningyu1 commented 5 years ago

返回处理的不彻底，#98 中修复了问题