[ 
https://issues.apache.org/jira/browse/DIRKRB-686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16326814#comment-16326814
 ] 

Frank Zeng edited comment on DIRKRB-686 at 1/16/18 7:03 AM:
------------------------------------------------------------

Refactor as [~coheig]'s suggestion.
 * Use KerberosTime.getTime() get the value of kerberos time.
 * Change "created_time", "expire_time" column type to BIGINT.


was (Author: zenglinx):
Refactor as [~colm] suggest.
 * Use KerberosTime.getTime() get the value of kerberos time.
 * Change "created_time", "expire_time" column type to BIGINT.

> Some refactor work for MySQL backend
> ------------------------------------
>
>                 Key: DIRKRB-686
>                 URL: https://issues.apache.org/jira/browse/DIRKRB-686
>             Project: Directory Kerberos
>          Issue Type: Improvement
>            Reporter: Frank Zeng
>            Assignee: Frank Zeng
>            Priority: Major
>             Fix For: 2.0.0
>
>         Attachments: DIRKRB-686-001.patch
>
>
> This is from [~coheig]'s suggestion:
> {quote}a) The mysql-backend uses different versions for junit + Apache 
> Directory as defined in the parent pom. So instead of 2.0.0-M23 for Apache 
> Directory you can use ${apacheds.version}. You can remove the junit version + 
> test scope because it's defined in the root pom under dependencyManagement. 
> b) The Apache Directory API is only used to call on GeneralizedTime, but this 
> pulls in a large number of dependencies in the distribution. Can we replace 
> GeneralizedTime with something else, or just pull GeneralizedTime into our 
> project directly? 
> c) I think the mysql-backend dependency should be commented out in the 
> kerby-dist pom.
> {quote}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to