-
Notifications
You must be signed in to change notification settings - Fork 465
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
[RPC] RPC支持加入错误类型error type
?
#5334
Comments
要的,后面想办法把我们的rpc库和官方的rpc库合并到一起去 |
lotus开发团队在json-rpc库中加入更详细的错误信息, 起因要从这个下面的问题说起: 由于在例如 而要实现更智能的处理上述问题, 更加准确的rpc错误信息是一个基础. 我们已经在自己fork的分支中做了大量的改动或优化, 统一RPC库时, 如何取舍我们'自己'的优化是一个问题. 但无论如何, 把我们自己的rpc库与lotus团队维护的rpc库保持一致, 是一种明智的做法. |
@hunjixin @simlecode 这个后来 |
看了下还没有 |
😂。。看来是没人合了。。能review里面,请求他们人来review么? |
这个官方还是没有合?尴尬。。。😂 |
RPC 的好像官方那边 @magik6k 还没有 review。这咋办? |
已经有 #5346 了。 |
为了支持 filecoin-project/lotus#9294 ,在
lotus
的master
中加入了 filecoin-project/lotus#9061 ,不确定Venus
是不是需要?另外,这个Lotus-miner
考虑在用这个方式解决daemon高可用的问题,和chain-co是2个方法。参考一下?关联:
The text was updated successfully, but these errors were encountered: