Skip to content

More debug implementations#387

Merged
frankmcsherry merged 1 commit intoTimelyDataflow:masterfrom
Kixiron:debug-handles
Jun 5, 2021
Merged

More debug implementations#387
frankmcsherry merged 1 commit intoTimelyDataflow:masterfrom
Kixiron:debug-handles

Conversation

@Kixiron
Copy link
Copy Markdown
Contributor

@Kixiron Kixiron commented May 2, 2021

Added a bunch of debug implementations and some minor clippy lints

Comment thread timely/src/dataflow/channels/pact.rs Outdated
Comment thread timely/src/dataflow/channels/pushers/tee.rs Outdated
Comment thread timely/src/dataflow/channels/pushers/tee.rs Outdated
Comment thread timely/src/dataflow/channels/pushers/tee.rs Outdated
Comment thread timely/src/dataflow/channels/pushers/tee.rs Outdated
Copy link
Copy Markdown
Member

@frankmcsherry frankmcsherry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These largely look great. I flagged a few formatting nits because we chatted about them. None of them are big deals, but each of them are "this will be harder for you to read now" changes which .. certainly aren't the end of the world.

I'm going to take a peek at a rustfmt.toml and see if I can put one together that I can live with, and then I won't end up saying things like this in the future. :D

@Kixiron
Copy link
Copy Markdown
Contributor Author

Kixiron commented Jun 5, 2021

I think I fixed everything, thanks!

Comment thread timely/src/dataflow/stream.rs Outdated
@frankmcsherry frankmcsherry merged commit 209af99 into TimelyDataflow:master Jun 5, 2021
@frankmcsherry
Copy link
Copy Markdown
Member

Thanks very much!

@Kixiron Kixiron deleted the debug-handles branch June 5, 2021 18:11
@github-actions github-actions Bot mentioned this pull request Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants