[
https://issues.apache.org/jira/browse/DIRKRB-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15231453#comment-15231453
]
Jiajia Li commented on DIRKRB-546:
--
Thanks for the updating, it looks good to me, +1.
> Add super class
[
https://issues.apache.org/jira/browse/DIRKRB-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15229791#comment-15229791
]
Kai Zheng commented on DIRKRB-546:
--
Thanks for the clarifying, Yan. Understood.
> Add super class to
[
https://issues.apache.org/jira/browse/DIRKRB-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15229767#comment-15229767
]
YanYan commented on DIRKRB-546:
---
Thanks for Kai's comments.
There already exist {{getAdminServerConfig}} and
[
https://issues.apache.org/jira/browse/DIRKRB-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15229611#comment-15229611
]
Kai Zheng commented on DIRKRB-546:
--
I think in this context, the word 'kdc' should be 'adminServer'.
>
[
https://issues.apache.org/jira/browse/DIRKRB-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15225893#comment-15225893
]
YanYan commented on DIRKRB-546:
---
Got it. I'll change the abstract super class into an interface.
> Add
[
https://issues.apache.org/jira/browse/DIRKRB-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15225833#comment-15225833
]
Jiajia Li commented on DIRKRB-546:
--
The getKdcRealm() and getKdcConfig() in ServerSetting can be the