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.

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

Change log source for followers to replication.follower #655

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

viktorerlingsson
Copy link
Member

WHAT is this pull request doing?

Changes the log source for replication followers from "self" to "replication.follower"

HOW can this pull request be tested?

Look at logs

@carlhoerberg
Copy link
Member

Do we want to drop the LavinMQ module completely instead? Or hack Log to drop it?

@spuun
Copy link
Member

spuun commented Apr 16, 2024

Do we want to drop the LavinMQ module completely instead? Or hack Log to drop it?

Imo we should keep it (or shorten it to lmq). But we need to decide on how we want the logging to work, if we wan't to be able to configure different levels for different sources or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants