mirror of
https://github.com/akvorado/akvorado.git
synced 2025-12-11 22:14:02 +01:00
outlet/kafka: delay shutdown when stopping worker after closing client
Some checks failed
CI / 🤖 Check dependabot status (push) Has been cancelled
CI / 🐧 Test on Linux (${{ github.ref_type == 'tag' }}, misc) (push) Has been cancelled
CI / 🐧 Test on Linux (coverage) (push) Has been cancelled
CI / 🐧 Test on Linux (regular) (push) Has been cancelled
CI / ❄️ Build on Nix (push) Has been cancelled
CI / 🍏 Build and test on macOS (push) Has been cancelled
CI / 🧪 End-to-end testing (push) Has been cancelled
CI / 🔍 Upload code coverage (push) Has been cancelled
CI / 🔬 Test only Go (push) Has been cancelled
CI / 🔬 Test only JS (${{ needs.dependabot.outputs.package-ecosystem }}, 20) (push) Has been cancelled
CI / 🔬 Test only JS (${{ needs.dependabot.outputs.package-ecosystem }}, 22) (push) Has been cancelled
CI / 🔬 Test only JS (${{ needs.dependabot.outputs.package-ecosystem }}, 24) (push) Has been cancelled
CI / ⚖️ Check licenses (push) Has been cancelled
CI / 🐋 Build Docker images (push) Has been cancelled
CI / 🐋 Tag Docker images (push) Has been cancelled
CI / 🚀 Publish release (push) Has been cancelled
Some checks failed
CI / 🤖 Check dependabot status (push) Has been cancelled
CI / 🐧 Test on Linux (${{ github.ref_type == 'tag' }}, misc) (push) Has been cancelled
CI / 🐧 Test on Linux (coverage) (push) Has been cancelled
CI / 🐧 Test on Linux (regular) (push) Has been cancelled
CI / ❄️ Build on Nix (push) Has been cancelled
CI / 🍏 Build and test on macOS (push) Has been cancelled
CI / 🧪 End-to-end testing (push) Has been cancelled
CI / 🔍 Upload code coverage (push) Has been cancelled
CI / 🔬 Test only Go (push) Has been cancelled
CI / 🔬 Test only JS (${{ needs.dependabot.outputs.package-ecosystem }}, 20) (push) Has been cancelled
CI / 🔬 Test only JS (${{ needs.dependabot.outputs.package-ecosystem }}, 22) (push) Has been cancelled
CI / 🔬 Test only JS (${{ needs.dependabot.outputs.package-ecosystem }}, 24) (push) Has been cancelled
CI / ⚖️ Check licenses (push) Has been cancelled
CI / 🐋 Build Docker images (push) Has been cancelled
CI / 🐋 Tag Docker images (push) Has been cancelled
CI / 🚀 Publish release (push) Has been cancelled
Flushing can take some time and we have an heartbeat to respect to commit offsets.
This commit is contained in:
@@ -66,7 +66,6 @@ func (c *realComponent) startOneWorker() error {
|
|||||||
Logger()
|
Logger()
|
||||||
defer func() {
|
defer func() {
|
||||||
logger.Info().Msg("stopping worker")
|
logger.Info().Msg("stopping worker")
|
||||||
shutdown()
|
|
||||||
|
|
||||||
// Allow a small grace time to commit uncommited work.
|
// Allow a small grace time to commit uncommited work.
|
||||||
ctx, cancelCommit := context.WithTimeout(context.Background(), 10*time.Second)
|
ctx, cancelCommit := context.WithTimeout(context.Background(), 10*time.Second)
|
||||||
@@ -74,8 +73,9 @@ func (c *realComponent) startOneWorker() error {
|
|||||||
if err := client.CommitMarkedOffsets(ctx); err != nil {
|
if err := client.CommitMarkedOffsets(ctx); err != nil {
|
||||||
logger.Err(err).Msg("cannot commit marked partition offsets")
|
logger.Err(err).Msg("cannot commit marked partition offsets")
|
||||||
}
|
}
|
||||||
|
|
||||||
client.CloseAllowingRebalance()
|
client.CloseAllowingRebalance()
|
||||||
|
|
||||||
|
shutdown()
|
||||||
}()
|
}()
|
||||||
|
|
||||||
for {
|
for {
|
||||||
|
|||||||
Reference in New Issue
Block a user