-
Notifications
You must be signed in to change notification settings - Fork 233
Exclude fingerprints from longreqs #5652
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
base: main
Are you sure you want to change the base?
Conversation
d002515 to
54a8e52
Compare
roborivers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
The first 10 failing tests are:
consumer_non_atomic_default_consumer_generated
truncatesc_offline_generated
insert_lots_ssl_generated
insert_lots
reco-ddlk-sql
guid
roborivers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
74a80f6 to
a570a79
Compare
Signed-off-by: Mark Hannum <mhannum@bloomberg.net>
a570a79 to
695fc38
Compare
roborivers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
The first 10 failing tests are:
logfill_logput_window_generated
consumer_non_atomic_default_consumer_generated
sc_downgrade
truncatesc_offline_generated
insert_lots_ssl_generated
insert_lots
roborivers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
The first 10 failing tests are:
logfill_logput_window_generated
logfill
consumer
consumer_non_atomic_default_consumer_generated
phys_rep_tiered_firstfile_generated
insert_lots_ssl_generated
insert_lots
reco-ddlk-sql
This PR adds the ability to exclude certain fingerprints from being reported in the longreqs log.