Skip to content

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

Closed
mmaxiaolei wants to merge 7 commits intoapache:masterfrom
mmaxiaolei:master
Closed

修改参数分组校验bug、ExecuteLimitFilter中统计信息错误,及其他一些优化#504
mmaxiaolei wants to merge 7 commits intoapache:masterfrom
mmaxiaolei:master

Conversation

@mmaxiaolei
Copy link
Copy Markdown

@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
Copy Markdown

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
Copy Markdown
Contributor

有挺多问题都已修复

@Authorlove
Copy link
Copy Markdown
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