Skip to content
This repository has been archived by the owner on Nov 29, 2024. It is now read-only.

Change column not found log message to debug #380

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Conversation

g-eoj
Copy link
Contributor

@g-eoj g-eoj commented Feb 9, 2024

We constantly see this warning for legitimate scoring requests.

It causes confusion when troubleshooting other issues.

I've never seen the warning help with actual issues.

Thus, we should not spam the logs with the warning as it causes more harm than good.

@g-eoj g-eoj requested a review from jackjii79 February 9, 2024 18:20
Copy link
Contributor

@jackjii79 jackjii79 left a comment

Choose a reason for hiding this comment

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

lgtm

@g-eoj g-eoj merged commit ed38903 into master Feb 9, 2024
2 of 10 checks passed
@g-eoj g-eoj deleted the joeg/log-clean-up branch February 9, 2024 18:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants