ipfs / go-graphsync

Initial Implementation Of GraphSync Wire Protocol
Other
100 stars 38 forks source link

Seeing "ProcessResponses: %!s(<nil>)" warning in Lotus markets logs using GS 0.9.1 #214

Closed stuberman closed 3 years ago

stuberman commented 3 years ago

Using Lotus markets version v1.11.3-rc1 graphsync v0.9.1 logs show these warnings during data transfers. It does not seem to affect the transfers.

2021-09-09T20:00:12.172Z WARN graphsync requestmanager/requestmanager.go:290 ProcessResponses: %!s() 2021-09-09T20:00:12.371Z WARN graphsync requestmanager/requestmanager.go:290 ProcessResponses: %!s() 2021-09-09T20:00:12.490Z WARN graphsync requestmanager/requestmanager.go:290 ProcessResponses: %!s() 2021-09-09T20:00:12.611Z WARN graphsync requestmanager/requestmanager.go:290 ProcessResponses: %!s() 2021-09-09T20:00:12.726Z WARN graphsync requestmanager/requestmanager.go:290 ProcessResponses: %!s()

Lotus client is Estuary

welcome[bot] commented 3 years ago

Thank you for submitting your first issue to this repository! A maintainer will be here shortly to triage and review. In the meantime, please double-check that you have provided all the necessary information to make this process easy! Any information that can help save additional round trips is useful! We currently aim to give initial feedback within two business days. If this does not happen, feel free to leave a comment. Please keep an eye on how this issue will be labeled, as labels give an overview of priorities, assignments and additional actions requested by the maintainers:

Finally, remember to use https://discuss.ipfs.io if you just need general support.

jennijuju commented 3 years ago

(lotus would love to have this wrapped properly sooner than later if possible! (and should the level be raised?

aarshkshah1992 commented 3 years ago

Fixed by https://github.com/ipfs/go-graphsync/pull/210.