ipfs / go-graphsync

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

Refactor: Use TaskQueue in ResponseManager #232

Closed hannahhoward closed 2 years ago

hannahhoward commented 3 years ago

What

We can simplify the ResponseManager by refactoring it to use the extracted TaskQueue logic. Even better if we can successfully extract the queryExecutor to its own module and unit test it (not as essential though), and then move instantiation of the query executor up to the top level module.

davidd8 commented 3 years ago

Assigning to @rvagg as a good starter task for graphsync.

rvagg commented 3 years ago

Working on this, but getting my head around what's going on in here and what needs to be replaced is requiring some learning investment.