Skip to content

Conversation

@enukane
Copy link
Contributor

@enukane enukane commented Jun 4, 2015

Fixed timeout parameter given to EV_SET(EVFILT_TIMER) in some BSD.

Only FreeBSD (or LINUX_SECOND?) has NOTE_SECONDS defined and specify seconds
in EV_SET(EVFILT_TIMER). Other BSD (NetBSD, OpenBSD, DragonFlyBSD) requires milliseconds.
This caused 1000 times faster timeout & flushing.

I've tested this patch in NetBSD environment (on my netbsd-work branch).

This patch fix timeout parameter given to EV_SET(EVFILT_TIMER) in BSD system
other than FreeBSD.  Only FreeBSD(or LINUX_SECOND) has NOTE_SECONDS defined and
specify seconds in EV_SET(EVFILT_TIMER). Other BSD expects milliseconds.

Tested in NetBSD environment.

Signed-off-by: enukane <[email protected]>
edsiper added a commit that referenced this pull request Jun 4, 2015
mk_core: fix timeout given to EVFILT_TIMER in some BSD
@edsiper edsiper merged commit 0a591a3 into fluent:master Jun 4, 2015
@edsiper
Copy link
Member

edsiper commented Jun 4, 2015

thanks

@prashantvicky prashantvicky mentioned this pull request Aug 28, 2018
fujimotos pushed a commit to fujimotos/fluent-bit that referenced this pull request Jul 22, 2019
output: new FlowCounter output documentation
nathalapooja pushed a commit to amazon-contributing/upstream-to-fluent-bit that referenced this pull request Oct 16, 2024
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.

2 participants