[
https://issues.apache.org/jira/browse/DIRKRB-205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14394173#comment-14394173
]
Lin Chen commented on DIRKRB-205:
---------------------------------
Hi Jiajia,
Thanks for your working on this. The work looks great. Some questions:
1. Should we add not-yet-commons-ssl dependency? I remove it and compile
successfully.
2. According to the patch, if my understanding is correct, when we add a
new identity to the backend, we need set principal first to create a zkNode for
the idnetity. Maybe we'd better not to depend on the interface calling order to
ensure the correctness. Is is better to make a new class, for example
{{IdentityZNode}}, which has a constructor with pricipal parameter? That can
ensure {{setPrincipal}} in the first place. Below is an example about
{{IdentityZNode}}:
{code}
public class IdentityZNode {
public IdentityZNode(String principalName) {
ZKUtil.createWithParents(....);
}
public void setKeyVersion(...) {...}
public void setKdcFlags(...) {...}
....
}
{code}
What do you think about this?
3. Are some methods in {{KerbyZNode}} like {{getPrincipalNameZnode}},
{{getKeyVersionZNode}} used on the class only? Is it better to make them to be
private?
I am not very familiar with zookeeper. If something is not correct, please
forgive me.
> Adding the identity to zookeeper backend
> ----------------------------------------
>
> Key: DIRKRB-205
> URL: https://issues.apache.org/jira/browse/DIRKRB-205
> Project: Directory Kerberos
> Issue Type: New Feature
> Reporter: Jiajia Li
> Assignee: Jiajia Li
> Attachments: DIRKRB-205-V1.patch, DIRKRB-205-V2.patch
>
>
> Implementing the feature of adding the identity to zookeeper backend.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
