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

KAFKA-3608: Fix ZooKeeper structures and output format in documentation #1257

Closed
wants to merge 1 commit into from

Conversation

vahidhashemian
Copy link
Contributor

No description provided.

@ijuma
Copy link
Contributor

ijuma commented May 8, 2016

We also have https://cwiki.apache.org/confluence/display/KAFKA/Kafka+data+structures+in+Zookeeper, @gwenshap which of the two copies do we want to maintain going forward? The one in the Git repo?

@gwenshap
Copy link
Contributor

Obviously the one in Git :)
Our structure docs should be versioned with the releases.

Any chance we could auto-generate those too? Maybe @vahidhashemian can do a follow-on? Or @granthenke, since he auto-generated the protocol?

@vahidhashemian
Copy link
Contributor Author

Not sure what is involved in auto-generate. If @granthenke sends me some pointers I can give it a try :)

@asfgit asfgit closed this in 18226ff May 10, 2016
asfgit pushed a commit that referenced this pull request May 10, 2016
Author: Vahid Hashemian <vahidhashemian@us.ibm.com>

Reviewers: Gwen Shapira

Closes #1257 from vahidhashemian/KAFKA-3608

(cherry picked from commit 18226ff)
Signed-off-by: Gwen Shapira <cshapi@gmail.com>
@gwenshap
Copy link
Contributor

For now, @vahidhashemian, can you just open a JIRA? Just so we won't lose track of the requirement?

@vahidhashemian
Copy link
Contributor Author

@gwenshap Sure, will do shortly.

gfodor pushed a commit to AltspaceVR/kafka that referenced this pull request Jun 3, 2016
Author: Vahid Hashemian <vahidhashemian@us.ibm.com>

Reviewers: Gwen Shapira

Closes apache#1257 from vahidhashemian/KAFKA-3608
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants