Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Speed up logging once again #9981

Merged
paritytech-processbot merged 12 commits into from
Oct 21, 2021
Merged

Speed up logging once again #9981

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
bda46c2
Update `tracing`-related dependencies
koute Oct 8, 2021
853214f
Enable `parking_lot` feature in `tracing-subscriber`
koute Oct 8, 2021
00e85ee
Add an asynchronous stderr logger
koute Oct 8, 2021
3981339
Make clippy happy
koute Oct 11, 2021
d6947b6
Add an integration test for the logger
koute Oct 11, 2021
5ae74c8
Refactor `test_logger_filters`'s subprocess machinery into a separate…
koute Oct 18, 2021
0fd0f24
Use a child process instead of hooking into stderr for the test
koute Oct 18, 2021
08103bd
Add a doc comment for `MakeStderrWriter`
koute Oct 18, 2021
bfa8f52
Move the initialization into the `MakeStderrWriter`'s constructor
koute Oct 21, 2021
be83a6c
Add an extra test case to trigger the logger's emergency flush mechanism
koute Oct 21, 2021
ce69d8c
Use the buffer's mutex for asynchronous flushes
koute Oct 21, 2021
949d1e1
Remove vestigial `nix` dependency from one of the previous commits
koute Oct 21, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Make clippy happy
  • Loading branch information
@koute
koute committed Oct 21, 2021
commit 39813393c114d3c12ba4c31cc5d0b9b2b36424e3
3 changes: 1 addition & 2 deletions client/tracing/src/logging/stderr_writer.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,8 +200,7 @@ impl Write for StderrWriter {
}

fn write_all(&mut self, input: &[u8]) -> Result<(), std::io::Error> {
self.write(input)?;
Ok(())
self.write(input).map(|_| ())
}

fn flush(&mut self) -> Result<(), std::io::Error> {
Expand Down