Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

instance termination plugin api #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

adyach
Copy link

@adyach adyach commented Dec 23, 2019

it should be possible to notify nakadi about coming instance termination, that it can release used resources and prepare itself for termination. A good example would be subscription connection termination in clean way, that consumer will not wait commit timeout

Copy link
Contributor

@rcillo rcillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

public interface TerminationService {

/**
* Registers lister to be executed once termination event happens.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/lister/listener/

@rcillo
Copy link
Contributor

rcillo commented Dec 23, 2019

👍

@rcillo
Copy link
Contributor

rcillo commented Dec 23, 2019

@adyach zappr seems confused. It's not reporting the status.

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