harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Regis Xu (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-5716) [classlib][jndi][ldap] - Refine implementation of LdapSchemaContextImpl
Date Fri, 11 Apr 2008 14:06:05 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-5716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12587978#action_12587978
] 

Regis Xu commented on HARMONY-5716:
-----------------------------------

Let me explain.

The file contains necessary data to construct LdapSchemaContextImpl instance for testing.

LdapSchemaContextImpl is constructed using schema data queried from LDAP server, then
it cached all scehma data in "schemaTable", it's fast for searching. 

It's best if we could test LdapSchemaContextImpl with "real data" returned from LDAP server.

So we serialize the "real" schemaTable to the file and deserialize it when testing, then we
got a 
"real" LdapSchemaContextImpl without connecting with server.

for 1)  we can construct the table from java code but the schema data is too large, i think
it's not reasonable
to put it in java file.

Maybe we could remove .ser from file name, which cause confusion with testing serialization
capabilities.

I hope i had myself explained clearly.




> [classlib][jndi][ldap] - Refine implementation of  LdapSchemaContextImpl
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-5716
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5716
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M5
>            Reporter: Regis Xu
>            Assignee: Tim Ellison
>             Fix For: 5.0M6
>
>         Attachments: HARMONY-5716.zip
>
>
> 1. extract parsing schema string returned from server to new class SchemaParser
> 2. refiene list, lookup, listBindings, getAttributes methods in LdapSchemaContextImpl
> 3. add new resource file TestLdapSchemaRoot.ser for testing ldap schema operations

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message