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

4.6.7 AbstractParser 加了2个 system.err.printlin 输出,关不掉,影响了压测性能 #232

Closed
xialonglee opened this issue May 13, 2021 · 6 comments

Comments

@xialonglee
Copy link

环境信息

  • 系统: Windows 10
  • JDK: 1.8.0_17
  • 数据库: MySQL 8
  • APIJSON: APIJSON 4.6.7

问题描述
这个4月7号的提交
8cf170c

在 apijson/orm/AbstractParser.java
加了2条 system.err.println 日志,关不掉

据我这边压测,发现这里占用了性能,而且关不掉,建议改成 log.d

System.err.println("\n\n\n\n<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<\n "
	+ TAG + ".DEBUG: " + requestMethod + "/parseResponse  request = \n" + requestString + "\n\n");

System.err.println(TAG + ".DEBUG: " + requestMethod + "/parseResponse return response = \n" + JSON.toJSONString(requestObject) + "\n\n");

错误信息

@TommyLemon
Copy link
Collaborator

感谢建议,这个信息很重要,后面会加一个变量来控制

@kxlv2000
Copy link
Contributor

您好,
请问问题解决了吗,可以允许我尝试修复此问题吗?

@TommyLemon
Copy link
Collaborator

TommyLemon commented May 19, 2021

您好,
请问问题解决了吗,可以允许我尝试修复此问题吗?

还没有,可以的,非常感谢。另外可以看看 【腾讯犀牛鸟开源人才培养计划】 哦,这可能对你是一个很好的机会
#229

@kxlv2000
Copy link
Contributor

好的,感谢回复,已查看【腾讯犀牛鸟开源人才培养计划】,我们考虑一下,同时我会推荐给身边的同学,感谢介绍!!

kxlv2000 added a commit to cs304-spring2021/APIJSON that referenced this issue May 19, 2021
kxlv2000 added a commit to cs304-spring2021/APIJSON that referenced this issue May 19, 2021
@TommyLemon
Copy link
Collaborator

TommyLemon commented Jun 30, 2021

感谢 @kxlv2000 解决了这个问题~
APIJSON 4.7.0 已发布
https://github.com/Tencent/APIJSON/releases/tag/4.7.0

chenyanlann pushed a commit to chenyanlann/APIJSON that referenced this issue Aug 1, 2021
chenyanlann pushed a commit to chenyanlann/APIJSON that referenced this issue Aug 1, 2021
@TommyLemon
Copy link
Collaborator

腾讯业务百万数据 6s 响应,APIJSON 性能优化背后的故事
https://my.oschina.net/tommylemon/blog/5375645

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

No branches or pull requests

3 participants