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

_thread_keepalive spams to much with timer logs #46

Open
2 tasks done
dolshevsk opened this issue Sep 29, 2023 · 2 comments
Open
2 tasks done

_thread_keepalive spams to much with timer logs #46

dolshevsk opened this issue Sep 29, 2023 · 2 comments

Comments

@dolshevsk
Copy link

dolshevsk commented Sep 29, 2023

Checklist

  • I have included information about relevant versions
  • I have verified that the issue persists when using the master branch of Mode.

Steps to reproduce

Hey there, I'm building quite intense streaming application and my logs just spammed with Timer _thread_keepalive-AIOKafkaConsumerThread woke up too late when the abs(drift) value between 0.3-0.5 what from my perspective isn't a big problem and doesn't need too much reaction.

Expected behavior

It would be great if max_drift could be configured somehow so it can be set to 1.0 when the lag is inappropriate, or produce a log only when self.drifting threshold is reached.

Actual behavior

Currently there is no option to configure either the max_drift or self.drifting thresholds.

Full traceback

[2023-09-29 15:41:39,425] [7] [INFO] Timer _thread_keepalive-AIOKafkaConsumerThread woke up too late, with a drift of +0.3595487903803587 runtime=6.0498714447021484e-06 sleeptime=1.3595487903803587

Versions

  • Python version: 3.11
  • Mode version: 0.3.5
  • Operating system: osx
@asaguaro1
Copy link

I am also having this issue. It consistently produces INFO logs that are notifying me of a < 1s drift with no other consequences that I can tell. Have you attempted any workarounds?

@wbarnha
Copy link
Member

wbarnha commented Mar 19, 2024

I agree, it is really a nuisance, let me see what I can do to give us the option to tell the application that drift is inevitable and that we don't care.

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

No branches or pull requests

3 participants