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

Consider adding forceSync=true for topic updates #114

Open
neoword opened this issue Feb 9, 2019 · 2 comments
Open

Consider adding forceSync=true for topic updates #114

neoword opened this issue Feb 9, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@neoword
Copy link
Contributor

neoword commented Feb 9, 2019

Desired Behavior

Add forceSync=true for updating of topic metadata.

[UPDATED]

This issue is a direct result of #52, in an attempt to resolve the "dead-end" a user will inevitably run into when they get an error from #52.

Benefits

@neoword neoword added the enhancement New feature or request label Feb 9, 2019
@OneCricketeer
Copy link
Contributor

OneCricketeer commented Apr 26, 2019

RE: "topic: " + topic " already exists but config is different than requested!"

Should the property instead be named something along the lines of

  • useExistingTopicConfig
  • allowConfigIfExists
  • allowConfigMismatch

@neoword
Copy link
Contributor Author

neoword commented May 7, 2019

Actually I am thinking this enhancement should sync the SR with the underlying broker metadata?
@Cricket007 do you agree?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants