Skip to content
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.

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

Some Exceptions Render ksqlDB Unusable #5896

Open
agavra opened this issue Jul 29, 2020 · 3 comments
Open

Some Exceptions Render ksqlDB Unusable #5896

agavra opened this issue Jul 29, 2020 · 3 comments

Comments

@agavra
Copy link
Contributor

agavra commented Jul 29, 2020

Describe the bug

Some bugs (e.g. #5882) make it so that the command topic is unusable. I'm not entirely certain how the command passed validation, but as soon as it is on the command topic it is no longer possible to start the ksqlDB server or submit any commands. The only option is to delete the command topic and recover it without the "bad" command.

To Reproduce

Repro using steps in #5882

Expected behavior

The command either never makes it onto the command topic, or we have some way to remediate that doesn't involve deleting the command topic.

Actual behavior

ksqlDB server becomes unusable.

Additional context

Original problem was observed on 5.3, it might be that this is no longer as bad of an issue but we should verify this.

@agavra agavra changed the title Some Exceptions Render the ksqlDB Unusable Some Exceptions Render ksqlDB Unusable Jul 29, 2020
@apurvam apurvam added the P0 Denotes must-have for a given milestone label Jul 29, 2020
@apurvam apurvam added this to the 0.12.0 milestone Jul 29, 2020
@agavra
Copy link
Contributor Author

agavra commented Jul 29, 2020

It looks like the case for #5882 does not make it onto the command topic, so it might not necessarily be possible for this case to come up in normal operation. The only time I can imagine this happening is if upgrading/downgrading ksqlDB versions causes an incompatible message to be on the command topic that causes the runner to crash (most issues won't). We can probably downgrade this from p0 cc @apurvam

@apurvam apurvam removed the P0 Denotes must-have for a given milestone label Jul 30, 2020
@apurvam
Copy link
Contributor

apurvam commented Jul 30, 2020

Thanks @agavra . Should we add a awaiting-info or some other label and drop from the milestone? It seems we can't actually reproduce this behavior right now.

@agavra
Copy link
Contributor Author

agavra commented Jul 30, 2020

I'm happy with that

@agavra agavra removed this from the 0.12.0 milestone Jul 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants