Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

修改参数分组校验bug、ExecuteLimitFilter中统计信息错误,及其他一些优化 #504

Closed
wants to merge 7 commits into from

Conversation

mmaxiaolei
Copy link

@mmaxiaolei mmaxiaolei commented Aug 8, 2017

  1. 对于内部类的获取使用了符号_,理应为$
    不过看JValidator.java的历史提交记录,原本就是用$连接的,后来改为_,理由为:

DUBBO-635 JValidator在类名生成的类名有$,有frozen class异常

DUBBO-635

实际测试中也未见frozen class异常

  1. ExecuteLimitFilter中统计调用信息的时候参数弄反了
  2. 优化了RpcStatus内存占用

@CLAassistant
Copy link

CLAassistant commented Aug 8, 2017

CLA assistant check
All committers have signed the CLA.

@mmaxiaolei mmaxiaolei changed the title 修改参数分组校验bug 修改参数分组校验bug、ExecuteLimitFilter中统计信息错误,及其他一些优化 Aug 9, 2017
@qinliujie qinliujie closed this Sep 27, 2017
@qinliujie
Copy link
Contributor

有挺多问题都已修复

@Authorlove
Copy link
Contributor

6ff60f5 这个提交会cherry-pick到master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants