Disable sync logging

[nicu-da]

[ci]

fixes #3521

Pull Request Checklist

Cluster Testing

• If a cluster test is required, comment /cluster_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
• If a hard-migration test is required (from the latest release), comment /hdm_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.

PR Guidelines

• Include any change that might be observable by our partners or affect their deployment in the release notes.
• Specify fixed issues with Fixes #n, and mention issues worked on using #n
• Include a screenshot for frontend-related PRs - see README or use your favorite screenshot tool

Merge Guidelines

• Make the git commit message look sensible when squash-merging on GitHub (most likely: just copy your PR description).

[moritzkiefer-da]

lgmt I suggest also putting it in the relesae notes, did you check what guarantees this gives around crashes? Do you lose logs just before you crash?

[nicu-da]

lgmt I suggest also putting it in the relesae notes, did you check what guarantees this gives around crashes? Do you lose logs just before you crash?

If you have a crash without proper shutdown logs are indeed lost that are not yet written. But during shutdown we handle closing the logger #3521 (comment)

[moritzkiefer-da]

sounds good thx