-
Notifications
You must be signed in to change notification settings - Fork 6
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
CBL-2835: enable core logging domains #60
Conversation
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.
@pasin What is the final word about the SQL domain? That would be the only change here that I could think of if SQL is too spammy.
I haven't looked it yet. @jayahariv Can you share two logs at the info level and at the verbose level to see how noisy it is? |
|
Merging. If it turns out that SQL is too verbose, please file a ticket to remove it. |
I'm not entirely clear on the goal for this ticket. Would you please check to be sure I've done the right thing?