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

升级到3.5.9还是不行,赶紧解决吧! #50

Open
berbatov001 opened this issue Nov 9, 2017 · 15 comments
Open

升级到3.5.9还是不行,赶紧解决吧! #50

berbatov001 opened this issue Nov 9, 2017 · 15 comments

Comments

@berbatov001
Copy link

berbatov001 commented Nov 9, 2017

public class ID {
@protobuf
private String id;
}
升级了3.5.9,编译时生成的代理ID$$JProtoBufClass.class大小是0K
退回到以前的3.4.1,就没这个问题。
但是3.4.1有性能问题,之前已经提过issue了,跑一段时间之后必须重启,否则吃光CPU 。
哎,你们能不能好好测一下再发布?别砸了百度的牌子。

@jhunters
Copy link
Collaborator

jhunters commented Nov 9, 2017

把你maven编译的插件版本发出一下,是不是使用的编译插件的问题?

@berbatov001
Copy link
Author

berbatov001 commented Nov 9, 2017

com.baidu jprotobuf-precompile-plugin 1.2.1 true com compile precompile

@jhunters
Copy link
Collaborator

jhunters commented Nov 9, 2017

plugin请升级到 1.2.6版本, 可能是plugin依赖的jprotobuf与rpc依赖的版本有冲突导致

@berbatov001
Copy link
Author

插件升级到1.2.6 编译报错,找不到com.sun.jersey.api.client.Client这个类

Failed to execute goal com.baidu:jprotobuf-precompile-plugin:1.2.6:precompile (default) on project id-generator-server: An exception occured while executing the Java class. null: InvocationTargetException: Scan entry error: EntryData{com.netflix.discovery.shared.transport.jersey.AbstractJerseyEurekaHttpClient'}; <--- java.lang.ClassNotFoundException: com.sun.jersey.api.client.Client: Lcom/sun/jersey/api/client/Client; -> [Help 1]

@jhunters
Copy link
Collaborator

jhunters commented Nov 9, 2017

plugin会扫描classpath的路径,你这个问题是你设置的扫描包的路径太大了。把你的path设置到你开发@protobuf注解的包就好了。 看一下下面 filterClassPackage 的配置

com.baidu
jprotobuf-precompile-plugin
1.2.5

true
com.baidu.xxxx



compile

precompile



@berbatov001
Copy link
Author

打包通过了,谢谢,有问题再请教你
你们这个版本是不是已经修复3.4.1那个吃光CPU的问题了?

@jhunters
Copy link
Collaborator

jhunters commented Nov 9, 2017

你说的吃光cpu还是内存?

@berbatov001
Copy link
Author

qq 20171109165617

打包过了,冒烟测试没过

@berbatov001
Copy link
Author

你好,昨天按照你说的方式测试了一下,还是有各种问题,但是回退到3.4.1版本就没问题。
能不能在3.4.1这个版本上打个补丁,专门修复好性能问题,可以吗?

@jhunters
Copy link
Collaborator

新版本还有什么问题? 另外 3.4.1的性能问题表现是什么样的,有没有具体的dupm线程?

@berbatov001
Copy link
Author

image
3.5.9启动时会这种警告,有影响吗?是需要设置哪里吗?

@berbatov001
Copy link
Author

image
3.4.1目前不定期会出现CPU升高的情况,因为3.4.1到3.5.9中间版本跨度比较大,可能引入了很多新的功能,不知是否出现其它问题。而3.4.1目前一直在用,除了运行一段时间CPU升高其它都还不错。
所以我想能不能麻烦在3.4.1的版本上发布一个小补丁。修复CPU升高的问题,其它的新功能可以不要。

@jhunters
Copy link
Collaborator

上面的警告不用关心,只是提示,没有error就可以了

@jhunters
Copy link
Collaborator

jhunters commented Nov 15, 2017

你上面使用3.5.9的问题,可以简单修改把 jprotobuf的版本手工指定为 1.10.7
<dependency> <groupId>com.baidu</groupId> <artifactId>jprotobuf</artifactId> <version>1.10.7</version> </dependency>

@jhunters
Copy link
Collaborator

cpu高性能目前只有你那边反馈,能否在cpu升高时,dump一下线程堆栈给我看一下

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

No branches or pull requests

2 participants