search

FISCO-BCOS / go-sdk

golang SDK of FISCO BCOS
Apache License 2.0
73 stars 58 forks source link

SubscribeEventLogs()方法订阅事件获取的数据不是有序的 #197

Open chaunsin opened 1 year ago

chaunsin commented 1 year ago

请问我在使用SubscribeEventLogs()方法订阅事件时，获取的数据不是按照区块顺序，通过多次测试发现每次获取的数据都是乱序的，请问能做优化处理么？

image
bxq2011hust commented 1 year ago

这个我确认下，SDK这边能否修改下做到

bxq2011hust commented 1 year ago

https://github.com/FISCO-BCOS/go-sdk/blob/16bfa5b5183dbd08503c4483ef27064ee1438be1/conn/channel.go#L1213

@chaunsin 我认为把上面这行代码前面的go删除应该就可以了，你测试下，如果不行就下面这行代码的go关键字也删除下 如果你测试ok，欢迎提个PR到master分支，这里放弃并行来换取事件通知有序可以接受

https://github.com/FISCO-BCOS/go-sdk/blob/16bfa5b5183dbd08503c4483ef27064ee1438be1/conn/channel.go#L1056

chaunsin commented 1 year ago

https://github.com/FISCO-BCOS/go-sdk/blob/16bfa5b5183dbd08503c4483ef27064ee1438be1/conn/channel.go#L1213

@chaunsin 我认为把上面这行代码前面的go删除应该就可以了，你测试下，如果不行就下面这行代码的go关键字也删除下 如果你测试ok，欢迎提个PR到master分支，这里放弃并行来换取事件通知有序可以接受

https://github.com/FISCO-BCOS/go-sdk/blob/16bfa5b5183dbd08503c4483ef27064ee1438be1/conn/channel.go#L1056

感谢回复,我这边尝试一下

chaunsin commented 1 year ago

https://github.com/FISCO-BCOS/go-sdk/blob/16bfa5b5183dbd08503c4483ef27064ee1438be1/conn/channel.go#L1213

@chaunsin 我认为把上面这行代码前面的go删除应该就可以了，你测试下，如果不行就下面这行代码的go关键字也删除下 如果你测试ok，欢迎提个PR到master分支，这里放弃并行来换取事件通知有序可以接受

https://github.com/FISCO-BCOS/go-sdk/blob/16bfa5b5183dbd08503c4483ef27064ee1438be1/conn/channel.go#L1056

感谢回复,我这边尝试一下

bxq2011hust commented 1 year ago

@chaunsin 问题解决了嘛？