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

阿里好意思拿着这么一个demo来作为sdk吗? #127

Open
xiedeacc opened this issue Jan 14, 2023 · 12 comments
Open

阿里好意思拿着这么一个demo来作为sdk吗? #127

xiedeacc opened this issue Jan 14, 2023 · 12 comments

Comments

@xiedeacc
Copy link

xiedeacc commented Jan 14, 2023

  1. 没必要自己整个ResourceGuard,并且放在example里面来误导人
  2. 能把rapidjson源码去掉吗?改成依赖引入,和自己引入的rapidjson冲突了,直接core了
  3. 单机多实例会导致core
@ty201115
Copy link

单机多实例遇到了,还好是放容器

@dengjfzh
Copy link

Listener的长轮询线程在进程退出时必须等当前请求超时,导致进程不能及时退出;
连接nacos时密码不对不会报错,而是直接 Assert Failed!

@yujitai
Copy link

yujitai commented Aug 10, 2023

希望阿里的人看下吧,c++ sdk 槽点太多了

@mopooo
Copy link

mopooo commented Sep 20, 2023

最新版本1.1,连接一个不需要用户名密码的server会直接crash,一看是rapidjson的assert出错,想catch都没机会。。。感情你们都没有起码的测试用例的吗?

@xiedeacc
Copy link
Author

xiedeacc commented Oct 9, 2023

再加一条,listener只能new,不能自己释放,垃圾阿里

@xiedeacc
Copy link
Author

xiedeacc commented Oct 9, 2023

再加一条,listener只能new,不能自己释放,垃圾阿里

所以一定不能把listener放到shared_ptr里面

@a839419160
Copy link

请教下。listener不能自己释放这个解决了没,必core

@xiedeacc
Copy link
Author

请教下。listener不能自己释放这个解决了没,必core

我的场景不会生成太多的listerner,所以就没有去释放了,有时间了给他们重写一个sdk吧

@zhwaaaaaa
Copy link

大佬 sdk 写出来没有,我想用

@huangxinze
Copy link

单机多实例遇到了,还好是放容器

单机多实例 会崩溃,请问您是怎么处理的?(如果不放在容器内的话)

@jackxuqq
Copy link

  1. 没必要自己整个ResourceGuard,并且放在example里面来误导人
  2. 能把rapidjson源码去掉吗?改成依赖引入,和自己引入的rapidjson冲突了,直接core了
  3. 单机多实例会导致core

-----请问单机多实例会导致core这个问题解决了没?

@jackxuqq
Copy link

@xiedeacc 大佬期待你重写的sdk的说

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

9 participants