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

Read Pieces by bypassing the scheduler and add extra logging for retrievals #6313

Closed
wants to merge 22 commits into from

Conversation

dirkmc
Copy link
Contributor

@dirkmc dirkmc commented May 21, 2021

Adds extra logging to retrieval deals on miner.

Branched from #6280

@aarshkshah1992 aarshkshah1992 changed the title add extra logging for retrievals Read Pieces by bypassing the scheduler and add extra logging for retrievals May 24, 2021
@magik6k magik6k added the team/ignite Issues and PRs being tracked by Team Ignite at Protocol Labs label Jun 3, 2021
@aarshkshah1992 aarshkshah1992 force-pushed the feat/pieceread-outside-scheduler branch from 50e023e to 21e6b50 Compare June 7, 2021 10:32
Base automatically changed from feat/pieceread-outside-scheduler to master June 7, 2021 16:11
@raulk
Copy link
Member

raulk commented Jun 8, 2021

@dirkmc I'm tentatively closing this PR because I suspect that the extra logging added here was also cherry-picked onto #6280 (already merged). If it's not the case, let's open a new PR with only the logging changes, since #6280 is already on master 🎉

@raulk raulk closed this Jun 8, 2021
@Kubuxu Kubuxu deleted the feat/pieceread-extra-logging branch November 25, 2021 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team/ignite Issues and PRs being tracked by Team Ignite at Protocol Labs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants