-
Notifications
You must be signed in to change notification settings - Fork 328
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
SAMZA-1330: Stand alone feature preview, known limitation. #220
Conversation
docs/startup/preview/index.md
Outdated
* The LocalApplicationRunner does not yet support the low level API. This means you cannot use StreamTask with LocalApplicationRunner. | ||
* Currently you can only run a single applicaiton with a single zookeeper. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this limitation was only for LinkedIn. Can't we workaround this issue by using a unique job.id
or app.id
??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, it still is a limitation. That's why it requires work around. Do you guys, want to add the workaround to the doc, or prefer remove it completely?
docs/startup/preview/index.md
Outdated
* The LocalApplicationRunner does not yet support the low level API. This means you cannot use StreamTask with LocalApplicationRunner. | ||
* Currently, you can only run a single applicaiton with a single zookeeper. To avoid this limitation, you need to make sure your app.id is unique. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can just say that the "app.id" must be unique for all applications using the zk cluster.
If not, typoe: "application" in the first line, and maybe say "single application at a time"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, this sounds simpler. Changed.
No description provided.