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.

Sign up for GitHub

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

Jump to bottom

Consider suppress-data-warnings option #40

Closed
alexdunnjpl opened this issue Jul 26, 2023 · 1 comment · Fixed by #45
Closed

Consider suppress-data-warnings option #40

alexdunnjpl opened this issue Jul 26, 2023 · 1 comment · Fixed by #45
Assignees
@alexdunnjpl
Labels
B14.0 i&t.skip Skip I&T of this task/ticket s.high High severity task

Comments

@alexdunnjpl
Copy link
Contributor

Missing/off-node data warning logs are enormously noisy, to the point where they can significantly extend execution time.

Consider implementing a method of suppressing these warnings, and/or aggregating them (like "n errors encountered, of types A, B, C")
@alexdunnjpl alexdunnjpl added B14.0 i&t.skip Skip I&T of this task/ticket task labels Jul 26, 2023
@alexdunnjpl alexdunnjpl self-assigned this Jul 26, 2023
@alexdunnjpl
Copy link
Contributor Author

The slowdown is large enough that this should be a priority.

@alexdunnjpl alexdunnjpl added the s.high High severity label Jul 26, 2023
This was referenced Jul 26, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexdunnjpl alexdunnjpl

Labels
B14.0 i&t.skip Skip I&T of this task/ticket s.high High severity task
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

40 add logging aggregation, LOGLEVEL env var, other logging tweaks NASA-PDS/registry-sweepers
2 participants
@jordanpadams @alexdunnjpl