Skip to content
This repository has been archived by the owner on Oct 25, 2023. It is now read-only.

Update kafka.init #13

Merged
merged 2 commits into from
Jan 25, 2019
Merged

Update kafka.init #13

merged 2 commits into from
Jan 25, 2019

Conversation

giena
Copy link
Contributor

@giena giena commented Jan 24, 2019

killproc: A bigger delay because the default delay is 3 seconds on CentOS

@id
Copy link
Owner

id commented Jan 25, 2019

Thanks!

@id
Copy link
Owner

id commented Jan 25, 2019

@giena
Copy link
Contributor Author

giena commented Jan 25, 2019

@id I think you are looking to the manpage of a killproc command. But in our case, killproc is a function contained in the /etc/rc.d/init.d/functions (that you use in kafka.init).
In the script "functions", you could read in the method killproc:
echo $"Usage: killproc [-p pidfile] [ -d delay] {program} [-signal]"
You will see that the default value is 3 seconds.
https://bash.cyberciti.biz/guide//etc/init.d/functions
You must respect the order of the parameters in the usage.
My kafka RPM uses your kafka.init. With 3 seconds my kafka instance does not have the time to flush his index. So at the startup, there was some corrupted index files. With a value of 600 seconds (ten minutes), it could be OK, because it's greater than the default value of timeout of systemd on CentOS 7 (five minutes). So systemd keeps the lead on the timeout. I think it's better.

@id
Copy link
Owner

id commented Jan 25, 2019

Thanks for the info.
I've been using https://github.com/id/kafka-el7-rpm for quite some time already, almost forgot about this one.

@id id merged commit ef03377 into id:master Jan 25, 2019
@giena
Copy link
Contributor Author

giena commented Jan 25, 2019

Thank you. I think i will do the same in the next version.

@giena giena deleted the patch-1 branch January 25, 2019 09:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants