-
Notifications
You must be signed in to change notification settings - Fork 2
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
multiple simultaneous errors cause intermixed stderr output #104
Comments
jtmoon79
added a commit
that referenced
this issue
May 8, 2023
Refactor channel data passing to have a consistent set of stages refelected in new enum `ChanDatum`. All processing threads must at least pass a `ChanDatum::FileInfo` with a `FileProcessingResultBlockZero`. All `FileProcessingResultBlockZero` printing are handled by the main printing thread instead of file processing threads. Pass a file Last Modified time (mtime) from file processing threads to main processing threads. Change all datas, `Sysline`, `Evtx`, `Utmpx`, `Journal` to have a `dt()` that returns `DateTimeL`. Previously the `Sysline.dt()` returned `DateTimeLOpt` which caused a fair amount of tedious code. Just assume a `Sysline` has a valid `DateTimeL`. Add necessary tests for `mtime()`. Issue #104 Issue #60
jtmoon79
added a commit
that referenced
this issue
May 8, 2023
Refactor channel data passing to have a consistent set of stages refelected in new enum `ChanDatum`. All processing threads must at least pass a `ChanDatum::FileInfo` with a `FileProcessingResultBlockZero`. All `FileProcessingResultBlockZero` printing are handled by the main printing thread instead of file processing threads. Pass a file Last Modified time (mtime) from file processing threads to main processing threads. Change all datas, `Sysline`, `Evtx`, `Utmpx`, `Journal` to have a `dt()` that returns `DateTimeL`. Previously the `Sysline.dt()` returned `DateTimeLOpt` which caused a fair amount of tedious code. Just assume a `Sysline` has a valid `DateTimeL`. Add necessary tests for `mtime()`. Issue #104 Issue #60
jtmoon79
added a commit
that referenced
this issue
May 8, 2023
Refactor channel data passing to have a consistent set of stages refelected in new enum `ChanDatum`. All processing threads must at least pass a `ChanDatum::FileInfo` with a `FileProcessingResultBlockZero`. All `FileProcessingResultBlockZero` printing are handled by the main printing thread instead of file processing threads. Pass a file Last Modified time (mtime) from file processing threads to main processing threads. Change all datas, `Sysline`, `Evtx`, `Utmpx`, `Journal` to have a `dt()` that returns `DateTimeL`. Previously the `Sysline.dt()` returned `DateTimeLOpt` which caused a fair amount of tedious code. Just assume a `Sysline` has a valid `DateTimeL`. Add necessary tests for `mtime()`. Issue #104 Issue #60
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
Multiple errors at nearly the same time result in an error message for each file path. The printed error messages
WARNING: no syslines found ...
are intermixed.Current behavior
For a path with many files that cannot be parsed, run
s4
This is rare but it does occur. The prior example was from FreeBSD 13.
Suggested behavior
Better coordinate writing to stderr among different threads. (I thought this was adequately coordinated).
The text was updated successfully, but these errors were encountered: