Skip to Content.
Sympa Menu

grouper-dev - Re: [grouper-dev] organize the grouper java package structure

Subject: Grouper Developers Forum

List archive

Re: [grouper-dev] organize the grouper java package structure


Chronological Thread 
  • From: "Tom Zeller" <>
  • To: "Chris Hyzer" <>
  • Cc: "" <>
  • Subject: Re: [grouper-dev] organize the grouper java package structure
  • Date: Wed, 9 Jul 2008 14:02:36 -0500
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version :content-type:references:x-google-sender-auth; b=K+8Hg6Tv/So2HnU1BVapssXe2X2kmm1YoppbLzGIB9snbe83IjoBoiGWoxjlVWTMb6 LLVRo6iFvOh/82ySSutKY31m07M9STZnySsauNeJdWDZF8+3bejICrzIkS8Deemj17g1 wY5Z/eT0mHjdMdov9ERpnhH9yQdkcnHNI8veA=

I like the idea of further package organization, although we'll probably need to agree on the details.

Perhaps while were changing lots of things, we should change even more, such as bringing gsh and usdu (and the loader ?) into the API ?

On Wed, Jul 9, 2008 at 1:33 PM, Chris Hyzer <> wrote:

Hey,

 

Im just thinking that once we roll out hooks, organizations will need to start navigating the grouper API.  Right now the base package has 135 java classes in it.  I think it would be a good idea to organize these into sub-packages.

 

e.g.

edu.internet2.middleware.grouper.beans (e.g. Group, Member)

edu.internet2.middleware.grouper.config (e.g. GrouperConfig)

edu.internet2.middleware.grouper.beans.finder (e.g. GroupFinder)

edu.internet2.middleware.grouper.exception (e.g. GrantPrivilegeException)

edu.internet2.middleware.grouper.filter (e.g. GroupAttributeFilter)

edu.internet2.middleware.grouper.log (e.g. EventLog)

edu.internet2.middleware.grouper.registry (e.g. RegistryInstall)

edu.internet2.middleware.grouper.validator (e.g. BaseQueryValidator)

edu.internet2.middleware.grouper.xml (e.g. XmlReader)

 

The upsides are:

1.       Grouper will be easier to maintain for the Grouper team

2.       Grouper hooks will be easier to use

3.       This is more in line with industry standard

 

The downsides are:

 

1.       CVS diffs harder (I was waiting for 1.4 to do major changes… plus there are so many diffs in files from the rework lately its less of a concern)

2.       Java security (package level wont protect methods as well, more public methods… I think this is ok since it is good hook implementers have more access, they will just need to be prepared to recompile every once and a while, we can help them with what to change as methods are changed or deprecated)


Again, after we roll out hooks, changes like this are harder to make.  So I suggest we do this for 1.4.  Any other API simplifications to make for 1.4?  Thoughts on this one?

 

Thanks,

Chris





Archive powered by MHonArc 2.6.16.

Top of Page