Skip to content

Return ConsumerStoppedEvent with Abnormal reason when MLC stopped abn… #3949

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Return ConsumerStoppedEvent with Abnormal reason when MLC stopped abn… #3949

wants to merge 1 commit into from
+32 −2

Conversation

AlamuriLokesh
Copy link

…ormally

artembilan
artembilan requested changes Jun 6, 2025
View reviewed changes
Copy link
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

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

Signed-off-by: AlamuriLokesh

Please, consider to fix your Git client to provide your official name.
According to DCO, the legal obligations for contributed code has to be from official name, not nicknames.

Otherwise we cannot review and accept your change.

@AlamuriLokesh AlamuriLokesh force-pushed the feature/GH-3948 branch 2 times, most recently from 515211d to e2626ed Compare June 6, 2025 14:54
@AlamuriLokesh
Copy link
Author 

Signed-off-by: AlamuriLokesh

Please, consider to fix your Git client to provide your official name. According to DCO, the legal obligations for contributed code has to be from official name, not nicknames.

Otherwise we cannot review and accept your change.

Could you please review now. I have updated my name as Alamuri Lokesh

sobychacko
sobychacko requested changes Jun 6, 2025
View reviewed changes
Copy link
Contributor

@sobychacko sobychacko left a comment

Choose a reason for hiding this comment

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

Some minor comments. We need to add reference docs for this since this is a new addition.

@sobychacko sobychacko added this to the 4.0.0-M3 milestone Jun 6, 2025
@AlamuriLokesh
Return ConsumerStoppedEvent with Abnormal reason when MLC stopped abn…
e14e6c2 
…ormally

Signed-off-by: Alamuri Lokesh <alamuri.lokesh@infosys.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sobychacko sobychacko

@artembilan artembilan

Assignees
No one assigned
Labels
type: enhancement
Projects
None yet
Milestone
4.0.0-M3
Development

Successfully merging this pull request may close these issues.
3 participants
@AlamuriLokesh @sobychacko @artembilan