Grouper Users - Open Discussion List

["Dr. Loris Bennett" <>]

Hi Chris,

I am running:

Debian 3.1 (Linux 2.6.22.1.dl380 #1 SMP Thu Aug 2 10:44:26 CEST 2007
GNU/Linux)
Postgres 8.4.2 (Debian 1:3.3.5-13)

I shall try out the workarounds and let you know how I get on.

Cheers

Loris


On Mon, 2009-02-09 at 10:03 -0500, Chris Hyzer wrote:
> Well, ddlutils is confused, it is reading jdbc metadata and it is saying 
> that there are multiple columns with the same name in the table...  
> I have seen postgres have more issues than oracle/mysql/hsql with 
> ddlutils...  anyways, I think you are stuck with one of the workarounds I 
> suggested. 
> Let me know the platform (linux?) and postgres version, and I will open a 
> bug and see if I can make progress on a longer term solution.
> 
> Thanks,
> Chris
> 
> 
> 
> > -----Original Message-----
> > From: Dr. Loris Bennett 
> > [mailto:]
> > Sent: Monday, February 09, 2009 9:55 AM
> > To: Chris Hyzer
> > Cc: Grouper Users Mailing List
> > Subject: RE: [grouper-users] RE: Grouper 1.4.0: Using hooks
> >
> > Here is the console output:
> >
> >         $ ./bin/gsh.sh -registry -check
> >         Using
> >         GROUPER_HOME: /home/testwww/src/grouper/grouper.binary.1.4.1
> >         Using
> >         GROUPER_CONF:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf
> >         Using JAVA: /home/testwww/java/jdk/bin/java
> >         using MEMORY: 64m-512m
> >         Grouper starting up: version: 1.4.1, build date: 2009/02/02
> >         14:43:32, env: testing
> >         grouper.properties read
> >         from:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/grouper.properties
> >         Grouper current directory
> >         is: /home/testwww/src/grouper/grouper.binary.1.4.1
> >         log4j.properties read
> >         from:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/log4j.properties
> >         Grouper is logging to
> >         file:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/logs/grouper_error.log,
> > at min level WARN for package: edu.internet2.middleware.grouper, based
> > on log4j.properties
> >         grouper.hibernate.properties:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/grouper.hibernate.p
> > roperties
> >         grouper.hibernate.properties:
> >         
> > grouper@jdbc:postgresql://identity-store.zedat.fu-
> > berlin.de:5713/testfurzdb
> >         sources.xml read
> >         from:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/sources.xml
> >         sources.xml groupersource id: g:gsa
> >         sources.xml jdbc source id:   jdbc:
> >         GrouperJdbcConnectionProvider
> >         (note, might need to type in your response multiple times (Java
> >         stdin is flaky))
> >         (note, you can whitelist or blacklist db urls and users in the
> >         grouper.properties)
> >         Are you sure you want to schemaexport all tables
> >         (dropThenCreate=F,writeAndRunScript=F) in db user 'grouper', db
> >         url
> >         'jdbc:postgresql://identity-store.zedat.fu-
> > berlin.de:5713/testfurzdb'? (y|n):
> >         y
> >         Continuing...
> >         Grouper ddl object type 'Grouper' has dbVersion: 12 and java
> >         version: 13
> >         org.apache.ddlutils.model.ModelException: There are multiple
> >         column with the name id in the table grouper_attributes
> >               at
> >
> > org.apache.ddlutils.model.Database.initialize(Database.java:295)
> >               at
> >
> > org.apache.ddlutils.platform.JdbcModelReader.getDatabase(JdbcModelReade
> > r.java:484)
> >               at
> >
> > org.apache.ddlutils.platform.PlatformImplBase.readModelFromDatabase(Pla
> > tformImplBase.java:1920)
> >               at
> >
> > edu.internet2.middleware.grouper.ddl.GrouperDdlUtils.bootstrapHelper(Gr
> > ouperDdlUtils.java:488)
> >               at
> >
> > edu.internet2.middleware.grouper.registry.RegistryInitializeSchema.main
> > (RegistryInitializeSchema.java:155)
> >               at sun.reflect.NativeMethodAccessorImpl.invoke0(Native
> > Method)
> >               at
> >
> > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.ja
> > va:39)
> >               at
> >
> > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccesso
> > rImpl.java:25)
> >               at java.lang.reflect.Method.invoke(Method.java:597)
> >               at
> >
> > edu.internet2.middleware.grouper.app.gsh.GrouperShell.handleSpecialCase
> > (GrouperShell.java:167)
> >               at
> >
> > edu.internet2.middleware.grouper.app.gsh.GrouperShell.main(GrouperShell
> > .java:108)
> >               at sun.reflect.NativeMethodAccessorImpl.invoke0(Native
> > Method)
> >               at
> >
> > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.ja
> > va:39)
> >               at
> >
> > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccesso
> > rImpl.java:25)
> >               at java.lang.reflect.Method.invoke(Method.java:597)
> >               at com.dawidweiss.invoker.Invoker.loader(Invoker.java:167)
> >               at com.dawidweiss.invoker.Invoker.main(Invoker.java:303)
> >         [com.dawidweiss.invoker.Invoker]
> >         edu.internet2.middleware.grouper.app.gsh.GrouperShell has
> > thrown
> >         an exception.
> >         java.lang.RuntimeException:
> >         java.lang.reflect.InvocationTargetException
> >               at
> >
> > edu.internet2.middleware.grouper.app.gsh.GrouperShell.handleSpecialCase
> > (GrouperShell.java:172)
> >               at
> >
> > edu.internet2.middleware.grouper.app.gsh.GrouperShell.main(GrouperShell
> > .java:108)
> >               at sun.reflect.NativeMethodAccessorImpl.invoke0(Native
> > Method)
> >               at
> >
> > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.ja
> > va:39)
> >               at
> >
> > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccesso
> > rImpl.java:25)
> >               at java.lang.reflect.Method.invoke(Method.java:597)
> >               at com.dawidweiss.invoker.Invoker.loader(Invoker.java:167)
> >               at com.dawidweiss.invoker.Invoker.main(Invoker.java:303)
> >         Caused by: java.lang.reflect.InvocationTargetException
> >               at sun.reflect.NativeMethodAccessorImpl.invoke0(Native
> > Method)
> >               at
> >
> > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.ja
> > va:39)
> >               at
> >
> > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccesso
> > rImpl.java:25)
> >               at java.lang.reflect.Method.invoke(Method.java:597)
> >               at
> >
> > edu.internet2.middleware.grouper.app.gsh.GrouperShell.handleSpecialCase
> > (GrouperShell.java:167)
> >               ... 7 more
> >         Caused by: java.lang.RuntimeException:
> >         org.apache.ddlutils.model.ModelException: There are multiple
> >         column with the name id in the table grouper_attributes
> >               at
> >
> > edu.internet2.middleware.grouper.registry.RegistryInitializeSchema.main
> > (RegistryInitializeSchema.java:176)
> >               ... 12 more
> >         Caused by: org.apache.ddlutils.model.ModelException: There are
> >         multiple column with the name id in the table
> > grouper_attributes
> >               at
> >
> > org.apache.ddlutils.model.Database.initialize(Database.java:295)
> >               at
> >
> > org.apache.ddlutils.platform.JdbcModelReader.getDatabase(JdbcModelReade
> > r.java:484)
> >               at
> >
> > org.apache.ddlutils.platform.PlatformImplBase.readModelFromDatabase(Pla
> > tformImplBase.java:1920)
> >               at
> >
> > edu.internet2.middleware.grouper.ddl.GrouperDdlUtils.bootstrapHelper(Gr
> > ouperDdlUtils.java:488)
> >               at
> >
> > edu.internet2.middleware.grouper.registry.RegistryInitializeSchema.main
> > (RegistryInitializeSchema.java:155)
> >               ... 12 more
> >
> > and here is grouper_error.log:
> >
> >         2009-02-09 15:50:26,049: [main] WARN
> >         ApiConfig.printConfigOnce(186) - Grouper starting up: version:
> >         1.4.1, build date: 2009/02/02 14:43:32, env: testing
> >         grouper.properties read
> >         from:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/grouper.properties
> >         Grouper current directory
> >         is: /home/testwww/src/grouper/grouper.binary.1.4.1
> >         log4j.properties read
> >         from:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/log4j.properties
> >         Grouper is logging to
> >         file:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/logs/grouper_error.log,
> > at min level WARN for package: edu.internet2.middleware.grouper, based
> > on log4j.properties
> >         grouper.hibernate.properties:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/grouper.hibernate.p
> > roperties
> >         grouper.hibernate.properties:
> >         
> > grouper@jdbc:postgresql://identity-store.zedat.fu-
> > berlin.de:5713/testfurzdb
> >         sources.xml read
> >         from:
> > /home/testwww/src/grouper/grouper.binary.1.4.1/conf/sources.xml
> >         sources.xml groupersource id: g:gsa
> >         sources.xml jdbc source id:   jdbc:
> >         GrouperJdbcConnectionProvider
> >         2009-02-09 15:50:35,664: [main] ERROR
> >         GrouperDdlUtils.bootstrapHelper(393) - Grouper ddl object type
> >         'Grouper' has dbVersion: 12 and java version: 13
> >
> > HTH
> >
> > Loris
> >
> > On Mon, 2009-02-09 at 09:45 -0500, Chris Hyzer wrote:
> > > How did you try to upgrade the DB?  With:
> > >
> > > gsh -registry -check -runscript
> > >
> > > Can you send me the entire log?
> > >
> > > As a workaround, you can either:
> > >
> > > 1. export to xml, recreate the db from scratch (delete data) with:
> > >
> > > gsh -registry -check -drop -runscript
> > >
> > > Then import the xml
> > >
> > > -or- 2. Change the grouper_stems stem name index to be unique (drop
> > and recreate index), and change the grouper_ddl row for grouper to be
> > version 13...
> > >
> > > Thanks,
> > > Chris
> > >
> > > > -----Original Message-----
> > > > From: Dr. Loris Bennett 
> > > > [mailto:]
> > > > Sent: Monday, February 09, 2009 7:55 AM
> > > > To: Chris Hyzer
> > > > Cc: Grouper Users Mailing List
> > > > Subject: RE: [grouper-users] RE: Grouper 1.4.0: Using hooks
> > > >
> > > > Hi Chris,
> > > >
> > > > Using your code I got everything working for a groupPostCommit
> > hook,
> > > > which is great.
> > > >
> > > > However, I can't try out the groupPostInsert hook with the CVS
> > version,
> > > > because when I try to update the database from 1.4.0, I get the
> > > > following error:
> > > >
> > > > Grouper ddl object type 'Grouper' has dbVersion: 12 and java
> > version:
> > > > 13
> > > >         org.apache.ddlutils.model.ModelException: There are
> > multiple
> > > > column with the name id in the table grouper_attributes
> > > >
> > > > The error message is misleading, since the table has in fact the
> > > > regular
> > > > 5 attributes - id, group_id, field_id, value,
> > hibernate_version_number.
> > > >
> > > > Any ideas?
> > > >
> > > > TIA
> > > >
> > > > Loris
> > > >
> > > > On Thu, 2009-02-05 at 16:21 -0500, Chris Hyzer wrote:
> > > > > Ok, the workaround isn’t working, lets fix the real issues
> > first...
> > > > >
> > > > > Those two bugs are fixed in the 1.4 branch, can you get latest
> > from
> > > > 1.4 branch?
> > > > >
> > > > > https://bugs.internet2.edu/jira/browse/GRP-219
> > > > > https://bugs.internet2.edu/jira/browse/GRP-220
> > > > >
> > > > > cvs 
> > > > > -d:pserver::/home/cvs/i2mi
> > > > >  login
> > > > > cvs 
> > > > > -d:pserver::/home/cvs/i2mi
> > export -r
> > > > GROUPER_1_4_BRANCH grouper
> > > > >
> > > > > Now we should be able to do this the original way, with a
> > postInsert
> > > > hook, and group.addType.
> > > > >
> > > > > Here is my testcase:
> > > > >
> > > > >   public void groupPostInsert(HooksContext hooksContext,
> > > > >       HooksGroupBean postInsertBean) {
> > > > >
> > > > >     super.groupPostInsert(hooksContext, postInsertBean);
> > > > >     try {
> > > > >       Group group = postInsertBean.getGroup();
> > > > >       GroupType fubGroup = GroupTypeFinder.find("fubGroup");
> > > > >       group.addType(fubGroup);
> > > > >       int sequenceNumber = 2;
> > > > >       group.setAttribute("gid", sequenceNumber + "");
> > > > >       group.store();
> > > > >     } catch (Exception e) {
> > > > >       throw new RuntimeException(e.getMessage(), e);
> > > > >     }
> > > > >   }
> > > > >
> > > > > Now, to get that "2", you want to use an oracle sequence?  You
> > should
> > > > be able to do something like this:
> > > > >
> > > > >       int sequenceNumber =
> > > > HibernateSession.bySqlStatic().select(int.class,
> > > > >         "select someSeq.nextval from dual");
> > > > >
> > > > > If you want to put it in a table, you can use hibernate, but
> > unless
> > > > you are doing a lot of sql, might just want to use sql...  if you
> > want
> > > > hibernate, let me know and I can make an example...
> > > > >
> > > > >       List<Object> params =
> > > > GrouperUtil.toList((Object)group.getUuid(), sequenceNumber);
> > > > >       HibernateSession.bySqlStatic().executeSql("insert into
> > > > some_table (col1, col2) values (?, ?)",
> > > > >           params);
> > > > >
> > > > >
> > > > > OK, all that being said, I think you have another problem... your
> > > > type has an extra attribute associated with it right (should only
> > have
> > > > gid, but it also has "id")???  You should be able to run this
> > query:
> > > > >
> > > > > select gt.name as type_name, gf.name as field_name,
> > > > > gf.is_nullable, gf.type as field_type, gf.read_privilege,
> > > > > gf.write_privilege, gt.is_assignable, gt.is_internal
> > > > >   from grouper_fields gf, grouper_types gt where
> > > > > gf.grouptype_uuid = gt.id
> > > > >
> > > > > You should see that fubGroup has one attribute named "gid".  If
> > you
> > > > see multiple attributes, you can delete those attributes (assuming
> > they
> > > > aren’t already being used :) ), with gsh...  I usually make my
> > group
> > > > types with java or gsh, not xml, but Im not sure what the issue
> > is...
> > > > >
> > > > >
> > > >
> > https://wiki.internet2.edu/confluence/display/GrouperWG/GrouperShell+(g
> > > > sh)
> > > > > typeDelField(type name, field name)
> > > > >
> > > > >
> > > > > Let me know the next step.
> > > > > Kind regards,
> > > > >
> > > > > Chris
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Dr. Loris Bennett 
> > > > > > [mailto:]
> > > > > > Sent: Thursday, February 05, 2009 6:14 AM
> > > > > > To: Chris Hyzer
> > > > > > Cc: Grouper Users Mailing List
> > > > > > Subject: RE: [grouper-users] RE: Grouper 1.4.0: Using hooks
> > > > > >
> > > > > > On Wed, 2009-02-04 at 11:38 -0500, Chris Hyzer wrote:
> > > > > > > Ok, this is a problem with grouper.  In addChildGroup in
> > Stem, it
> > > > > > assume that the types aren’t saved yet right after the group
> > insert
> > > > > > (since hooks didn’t exist back then, it was a safe assumption).
> > > > > > >
> > > > > > > I added a bug:
> > > > > > >
> > > > > > > https://bugs.internet2.edu/jira/browse/GRP-219
> > > > > > >
> > > > > > > We will be able to fix this no problem.
> > > > > > >
> > > > > > > In the meantime, you have two options:
> > > > > > >
> > > > > > > 1. Instead of adding a type, add it to the list of types that
> > the
> > > > > > calling method will add:
> > > > > > >
> > > > > > > FROM:
> > > > > > > g.addType(fubGroup);
> > > > > > >
> > > > > > > TO:
> > > > > > > g.getTypesDb().add(fubGroup);
> > > > > > >
> > > > > > > This seems like the best temporary workaround until we fix
> > the
> > > > real
> > > > > > issue.  Though this using an internal method, so you should
> > > > definitely
> > > > > > switch it out when you get a fixed group.  I tried it and it
> > works.
> > > > > >
> > > > > > This gives me the error:
> > > > > >
> > > > > >         2009-02-05 09:11:54,097: [http-130.133.2.116-9080-
> > > > Processor21]
> > > > > >         ERROR GrouperCapableAction.execute(281) -
> > > > > >         java.lang.RuntimeException: edu.internet2.mid
> > > > > >         dleware.grouper.exception.SchemaException: invalid
> > group
> > > > type:
> > > > > >         for group name: test:abc123,
> > > > > >         GroupType[creatorUuid=b0bb6118-c6b9-4d37-8257-
> > 61dcee19b4c
> > > > > >
> > > > > >
> > > >
> > 6,createTime=1231229012373,fields=12,isAssignable=false,isInternal=fals
> > > > > > e,name=base,uuid=fc8eff6c-6812-4a10-8891-
> > 133df83bfbb5]:extension
> > > > > >
> > > > > > The UI says that the group was saved successfully, but also
> > that an
> > > > > > error occurred. When I click on the group in the UI to get the
> > > > group
> > > > > > summary, I get the standard error message.
> > > > > >
> > > > > > So I tried the second method (see below) ...
> > > > > >
> > > > > > > ################################
> > > > > > >
> > > > > > > -or- 2.  Do a postCommitInsert hook.  This will not be in the
> > > > same
> > > > > > transaction, but all the work of adding the group will be done,
> > so
> > > > you
> > > > > > are safe to add a type.
> > > > > > >  Again, if you pick this, then you might want to switch it
> > back
> > > > when
> > > > > > you get a fixed grouper. I tried this and it didn’t commit the
> > > > changes
> > > > > > (something about post
> > > > > > > commit puts it in a transaction that doesn’t get committed).
> > I
> > > > made
> > > > > > another bug:
> > > > > > >
> > > > > > > https://bugs.internet2.edu/jira/browse/GRP-220
> > > > > > >
> > > > > > > The workaround here is to just make a new transaction that
> > will
> > > > > > commit when done.  I tested it and it works:
> > > > > > >
> > > > > > > FROM:
> > > > > > >     public void groupPostInsert(HooksContext hooksContext,
> > > > > > HooksGroupBean postInsertBean) {
> > > > > > >
> > > > > > > TO:
> > > > > > >     public void groupPostCommitInsert(HooksContext
> > hooksContext,
> > > > > > HooksGroupBean postInsertBean) {
> > > > > > >
> > > > > > > FROM:
> > > > > > >       g.addType(fubGroup);
> > > > > > >
> > > > > > > TO:
> > > > > > >       final Group G = g;
> > > > > > >       final GroupType FUB_GROUP = fubGroup;
> > > > > > >       HibernateSession.callbackHibernateSession(
> > > > > > >           GrouperTransactionType.READ_WRITE_NEW, new
> > > > > > HibernateHandler() {
> > > > > > >
> > > > > > >             public Object callback(HibernateSession
> > > > hibernateSession)
> > > > > > >                 throws GrouperDAOException {
> > > > > > >               try {
> > > > > > >                 G.addType(FUB_GROUP);
> > > > > > >               } catch (Exception e) {
> > > > > > >                 throw new RuntimeException(e);
> > > > > > >               }
> > > > > > >               return null;
> > > > > > >             }
> > > > > > >
> > > > > > >           });
> > > > > > >
> > > > > >
> > > > > > This works. The group type is added with the attributes
> > > > > >
> > > > > >         gid <empty>
> > > > > >         id <same as uuid>
> > > > > >
> > > > > > However, my definition of the group type looks like
> > > > > >
> > > > > >         <registry>
> > > > > >           <metadata>
> > > > > >             <groupTypesMetaData>
> > > > > >               <groupTypeDef name='fubGroup'>
> > > > > >                 <field name='gid' required='false'
> > type='attribute'
> > > > > >         readPriv='read' writePriv='admin'/>
> > > > > >               </groupTypeDef>
> > > > > >             </groupTypesMetaData>
> > > > > >           </metadata>
> > > > > >         </registry>
> > > > > >
> > > > > > so where does 'id' come from?
> > > > > >
> > > > > > > Let me know if you need more help.
> > > > > >
> > > > > > Another thing I am not sure about is how I would do an insert
> > into
> > > > a
> > > > > > table such as "fub_group_ids" to get a gid from a sequence
> > which I
> > > > can
> > > > > > then write into the 'gid' field of my customised type. I am not
> > too
> > > > > > familiar with hibernate, but assume this would be the way to
> > go,
> > > > right?
> > > > > >
> > > > > > Thanks for all your help,
> > > > > >
> > > > > > Loris
> > > > > >
> > > > > > > Kind regards,
> > > > > > > Chris
> > > > > > >
> > > > > > > > -----Original Message-----
> > > > > > > > From: Dr. Loris Bennett 
> > > > > > > > [mailto:]
> > > > > > > > Sent: Wednesday, February 04, 2009 6:08 AM
> > > > > > > > To: Chris Hyzer
> > > > > > > > Cc: Grouper Users Mailing List
> > > > > > > > Subject: RE: [grouper-users] RE: Grouper 1.4.0: Using hooks
> > > > > > > >
> > > > > > > > Hi Chris,
> > > > > > > >
> > > > > > > > I tried the failsafe call, but got the same errors. In the
> > UI:
> > > > > > > >
> > > > > > > >         Error: Could not create group. Error is 'cannot
> > create
> > > > > > child
> > > > > > > >         group: Problem create child stem:
> > > > > > > >         Stem[displayName=test,name=test,uuid=029bc9c2-50bd-
> > > > 4122-
> > > > > > 891c-
> > > > > > > > ffdf252744b8,creator=fd703376-4f16-47df-b707-
> > > > > > > > 64d8c5953d04,modifier=fd703376-4f16-47df-b707-
> > 64d8c5953d04],
> > > > child:
> > > > > > > > Group[name=test:xxx,uuid=d559eccd-9be2-4555-ac1a-
> > 40ee72bb106e],
> > > > > > > > memberDto: 'd559eccd-9be2-4555-ac1a-
> > > > 40ee72bb106e'/'group'/'g:gsa',
> > > > > > > > Problem in HibernateSession: HibernateSession: isNew:
> > false,
> > > > > > > > isReadonly: false, grouperTransactionType: READ_WRITE_NEW,
> > > > > > Exception in
> > > > > > > > uniqueResult: (class java.lang.Object), ByHqlStatic, query:
> > > > 'select
> > > > > > > > m.id from Member as m where m.uuid = :uuid', cacheable:
> > false,
> > > > > > > > cacheRegion:
> > > > > > > >
> > > > > >
> > > >
> > edu.internet2.middleware.grouper.internal.dao.hib3.Hib3MemberDAO.Exists
> > > > > > > > , tx type: nullBind var[0]: 'Param (class
> > java.lang.String):
> > > > > > 'uuid'-
> > > > > > > > >'90e883dd-326e-40f1-9416-805d2770a9d6', , Problem in
> > > > > > HibernateSession:
> > > > > > > > HibernateSession: isNew: false, isReadonly: false,
> > > > > > > > grouperTransactionType: READ_WRITE_NEW'.
> > > > > > > >
> > > > > > > > I have attached an extract from the grouper log and the
> > code.
> > > > > > > >
> > > > > > > > Thanks for all your help.
> > > > > > > >
> > > > > > > > Loris
> > > > > > > >
> > > > > > > >
> > > > > > > > On Tue, 2009-02-03 at 23:32 -0500, Chris Hyzer wrote:
> > > > > > > > > Ive been thinking about your issue, and if your question
> > is,
> > > > why
> > > > > > > > should you use the failsafe group type call since the type
> > > > hasn’t
> > > > > > been
> > > > > > > > applied yet, it might be because when you add a type to a
> > > > group, it
> > > > > > > > might update the group lastUpdated col, and invoke the
> > update
> > > > hook
> > > > > > > > again.  Then, the second time, it does have that type, and
> > you
> > > > cant
> > > > > > add
> > > > > > > > it again.  This is something we are exploring for 1.5 (not
> > > > setting
> > > > > > the
> > > > > > > > Group lastUpdated for each peripheral operation like
> > > > memberships,
> > > > > > > > types, etc).  However, for 1.4, the group object might be
> > saved
> > > > for
> > > > > > any
> > > > > > > > related change, which fires the hooks again...  so if you
> > use
> > > > the
> > > > > > > > failsafe method per one of my previous emails, then the
> > second
> > > > time
> > > > > > the
> > > > > > > > hook is called, it doesn’t fail.
> > > > > > > > >
> > > > > > > > > Hope this helps, if not, send your code and description
> > again
> > > > and
> > > > > > I
> > > > > > > > > will take a look.  :)
> > > > > > > > >
> > > > > > > > > Kind regards,
> > > > > > > > > Chris
> > > > > > > > >
> > > > > > > > > > -----Original Message-----
> > > > > > > > > > From: Dr. Loris Bennett 
> > > > > > > > > > [mailto:loris.bennett@fu-
> > berlin.de]
> > > > > > > > > > Sent: Tuesday, February 03, 2009 9:37 AM
> > > > > > > > > > To: Chris Hyzer
> > > > > > > > > > Cc: Grouper Users Mailing List
> > > > > > > > > > Subject: RE: [grouper-users] RE: Grouper 1.4.0: Using
> > hooks
> > > > > > > > > >
> > > > > > > > > > Hi Chris,
> > > > > > > > > >
> > > > > > > > > > I don't understand what is going on here. If I create a
> > new
> > > > > > group
> > > > > > > > > > via the UI without selecting any custom type, the code
> > > > > > > > > >
> > > > > > > > > >         Group g = postInsertBean.getGroup();
> > > > > > > > > >         Set<GroupType> types = g.getTypes();
> > > > > > > > > >
> > > > > > > > > > tells me that the new group has just one type, namely
> > > > 'base'. I
> > > > > > > > then
> > > > > > > > > > get the error
> > > > > > > > > >
> > > > > > > > > >         ERROR JDBCExceptionReporter.logExceptions(78) -
> > > > ERROR:
> > > > > > > > > > duplicate
> > > > > > > > > >         key violates unique constraint
> > > > > > > > "grouptypetyple_grouptype_idx"
> > > > > > > > > >
> > > > > > > > > > But how can the group already have the type at this
> > point?
> > > > The
> > > > > > > > > > following error appears in the UI:
> > > > > > > > > >
> > > > > > > > > >         Error: Could not create group. Error is 'cannot
> > > > create
> > > > > > > > child
> > > > > > > > > >         group: Problem create child stem:
> > > > > > > > > >
> > > > > > > > > > Stem[displayName=test,name=test,uuid=029bc9c2-50bd-
> > 4122-
> > > > 891c-
> > > > > > > > > > ffdf252744b8,creator=fd703376-4f16-47df-b707-
> > > > > > > > > > 64d8c5953d04,modifier=fd703376-4f16-47df-b707-
> > > > 64d8c5953d04],
> > > > > > child:
> > > > > > > > > > Group[name=test:xyz,uuid=dc2a2fa6-4fad-446e-bf5c-
> > > > d08dba8d0b33],
> > > > > > > > > > memberDto: 'dc2a2fa6-4fad-446e-bf5c-
> > > > > > d08dba8d0b33'/'group'/'g:gsa',
> > > > > > > > > > Problem in HibernateSession: HibernateSession: isNew:
> > > > false,
> > > > > > > > > > isReadonly: false, grouperTransactionType:
> > READ_WRITE_NEW,
> > > > > > > > Exception
> > > > > > > > > > in
> > > > > > > > > > uniqueResult: (class java.lang.Object), ByHqlStatic,
> > query:
> > > > > > 'select
> > > > > > > > > > m.id from Member as m where m.uuid = :uuid', cacheable:
> > > > false,
> > > > > > > > > > cacheRegion:
> > > > > > > > > >
> > > > > > > >
> > > > > >
> > > >
> > edu.internet2.middleware.grouper.internal.dao.hib3.Hib3MemberDAO.Exi
> > > > > > > > > > sts , tx type: nullBind var[0]: 'Param (class
> > > > > > java.lang.String):
> > > > > > > > > > 'uuid'-
> > > > > > > > > > >'b412a97d-080a-4b68-a856-5a237a041e06', , Problem in
> > > > > > > > HibernateSession:
> > > > > > > > > > HibernateSession: isNew: false, isReadonly: false,
> > > > > > > > > > grouperTransactionType: READ_WRITE_NEW'.
> > > > > > > > > >
> > > > > > > > > > Any ideas what I am doing wrong?
> > > > > > > > > >
> > > > > > > > > > Cheers
> > > > > > > > > >
> > > > > > > > > > Loris
> > > > > > > > > >
> > > > > > > > > > On Fri, 2009-01-30 at 15:36 -0500, Chris Hyzer wrote:
> > > > > > > > > > > That index "grouptypetyple_grouptype_idx" is a unique
> > > > index
> > > > > > on
> > > > > > > > > > > grouper_group_types cols: group_uuid and type_uuid.
> > > > > > > > > > >
> > > > > > > > > > > So that error means that the group already has the
> > type.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > I have also found it unnatural that the default
> > Grouper
> > > > API
> > > > > > > > > > > behavior is to throw exceptions if the task is
> > already
> > > > done.
> > > > > > > > E.g.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > group.addType(someType);
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Will throw an exception if the type is already
> > there... I
> > > > > > have
> > > > > > > > > > > been slowly overloading these types of methods with
> > their
> > > > > > safe
> > > > > > > > > > > equivalents...  so if you change it to:
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > //false means don’t throw exception if type is
> > already
> > > > > > assigned
> > > > > > > > > > >
> > > > > > > > > > > group.addType(someType, false);
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > It should do the trick.  Let me know if not.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Thanks,
> > > > > > > > > > >
> > > > > > > > > > > Chris
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Ps.   Btw, web services do not behave that way...
> > e.g. if
> > > > you
> > > > > > add
> > > > > > > > a
> > > > > > > > > > > member to a group that already has the member, it
> > will be
> > > > a
> > > > > > no-
> > > > > > > > op,
> > > > > > > > > > > return SUCCESS, and let you know exactly what
> > happened
> > > > (e.g.
> > > > > > > > > > > success, but it already had the assignment)
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > -----Original Message-----
> > > > > > > > > > >
> > > > > > > > > > > > From: Dr. Loris Bennett 
> > > > > > > > > > > > [mailto:loris.bennett@fu-
> > > > berlin.de]
> > > > > > > > > > >
> > > > > > > > > > > > Sent: Friday, January 30, 2009 8:27 AM
> > > > > > > > > > >
> > > > > > > > > > > > To: Chris Hyzer
> > > > > > > > > > >
> > > > > > > > > > > > Cc: Grouper Users Mailing List
> > > > > > > > > > >
> > > > > > > > > > > > Subject: RE: [grouper-users] RE: Grouper 1.4.0:
> > Using
> > > > hooks
> > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > However, when I try to add the type in
> > groupPostInsert,
> > > > I
> > > > > > get
> > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > >         ERROR
> > JDBCExceptionReporter.logExceptions(78) -
> > > > > > ERROR:
> > > > > > > > > > >
> > > > > > > > > > > > duplicate
> > > > > > > > > > >
> > > > > > > > > > > >         key violates unique constraint
> > > > > > > > > > > "grouptypetyple_grouptype_idx"
> > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > Any ideas?
> > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > Loris
> > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > On Fri, 2009-01-30 at 07:34 -0500, Chris Hyzer
> > wrote:
> > > > > > > > > > >
> > > > > > > > > > > > > Good catch!  Thanks!
> > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > -----Original Message-----
> > > > > > > > > > >
> > > > > > > > > > > > > > From: Dr. Loris Bennett
> > [mailto:loris.bennett@fu-
> > > > > > berlin.de]
> > > > > > > > > > >
> > > > > > > > > > > > > > Sent: Friday, January 30, 2009 7:31 AM
> > > > > > > > > > >
> > > > > > > > > > > > > > To: Chris Hyzer
> > > > > > > > > > >
> > > > > > > > > > > > > > Cc: Grouper Users Mailing List
> > > > > > > > > > >
> > > > > > > > > > > > > > Subject: RE: [grouper-users] RE: Grouper 1.4.0:
> > > > Using
> > > > > > hooks
> > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > OK, I see what I am doing wrong. I have the
> > addType
> > > > in
> > > > > > my
> > > > > > > > > > >
> > > > > > > > > > > > > > groupPreInsert, whereas the group must already
> > > > exist
> > > > > > before
> > > > > > > > > > > > > > I
> > > > > > > > > > > can
> > > > > > > > > > >
> > > > > > > > > > > > add
> > > > > > > > > > >
> > > > > > > > > > > > > > anything to it.
> > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > On Fri, 2009-01-30 at 11:39 +0100, Dr. Loris
> > > > Bennett
> > > > > > wrote:
> > > > > > > > > > >
> > > > > > > > > > > > > > > Hi Chris,
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > Using
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > >         fubGroup =
> > > > > > GroupTypeFinder.find(groupTypeName);
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > I can get my custom group type, but when I
> > try to
> > > > add
> > > > > > > > this
> > > > > > > > > > > with
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > >         g.addType(fubGroup);
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > I get the error:
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > >         ERROR
> > > > JDBCExceptionReporter.logExceptions(78)
> > > > > > -
> > > > > > > > > > ERROR:
> > > > > > > > > > >
> > > > > > > > > > > > insert
> > > > > > > > > > >
> > > > > > > > > > > > > > or
> > > > > > > > > > >
> > > > > > > > > > > > > > >         update on table
> > "grouper_groups_types"
> > > > > > violates
> > > > > > > > > > > foreign
> > > > > > > > > > >
> > > > > > > > > > > > key
> > > > > > > > > > >
> > > > > > > > > > > > > > >         constraint
> > "fk_groups_types_group_uuid"
> > > > > > > > > > >
> > > > > > > > > > > > > > >           Detail: Key
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > (group_uuid)=(6ea9f650-cdc8-4e69-93df-
> > > > 5b35a87351fe)
> > > > > > > > > > is
> > > > > > > > > > >
> > > > > > > > > > > > not
> > > > > > > > > > >
> > > > > > > > > > > > > > >         present in table "grouper_groups".
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > Am I doing something wrong here?
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > TIA
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > Loris
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > On Thu, 2009-01-29 at 03:21 -0500, Chris
> > Hyzer
> > > > wrote:
> > > > > > > > > > >
> > > > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > > > So the pre-insert would also set the
> > custom
> > > > > > > > attribute,
> > > > > > > > > > > right?
> > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > > OK, didn’t know we were talking about a
> > custom
> > > > > > > > attribute.
> > > > > > > > > > > It’s
> > > > > > > > > > >
> > > > > > > > > > > > a
> > > > > > > > > > >
> > > > > > > > > > > > > > little tricky since the set of attributes is
> > known
> > > > when
> > > > > > the
> > > > > > > > > > > group
> > > > > > > > > > >
> > > > > > > > > > > > SQL
> > > > > > > > > > >
> > > > > > > > > > > > > > is written, but you are making changes (I guess
> > to
> > > > that
> > > > > > > > > > > attribute
> > > > > > > > > > >
> > > > > > > > > > > > set).
> > > > > > > > > > >
> > > > > > > > > > > > > > I would think a post insert is the best bet,
> > and
> > > > you
> > > > > > don’t
> > > > > > > > > > > > > > need
> > > > > > > > > > > the
> > > > > > > > > > >
> > > > > > > > > > > > > > pre, but play around and try different things,
> > and
> > > > if
> > > > > > you
> > > > > > > > > > > > > > cant
> > > > > > > > > > > get
> > > > > > > > > > >
> > > > > > > > > > > > > > anything to work let me know.
> > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > > Chris
> > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > > > --
> > > > > > > > > > >
> > > > > > > > > > > > > > Dr. Loris Bennett (Mr.)
> > > > > > > > > > >
> > > > > > > > > > > > > > Freie Universität Berlin
> > > > > > > > > > >
> > > > > > > > > > > > > > ZEDAT - Zentraleinrichtung für
> > Datenverarbeitung /
> > > > > > Computer
> > > > > > > > > > > Center
> > > > > > > > > > >
> > > > > > > > > > > > > > Compute & Media Service
> > > > > > > > > > >
> > > > > > > > > > > > > > Fabeckstr. 32, Room 221
> > > > > > > > > > >
> > > > > > > > > > > > > > D-14195 Berlin
> > > > > > > > > > >
> > > > > > > > > > > > > > Tel ++49 30 838 51024
> > > > > > > > > > >
> > > > > > > > > > > > > > Fax ++49 30 838 56721
> > > > > > > > > > >
> > > > > > > > > > > > > > Email 
> > > > > > > > > > > > > > 
> > > > > > > > > > >
> > > > > > > > > > > > > > Web www.zedat.fu-berlin.de
> > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > > --
> > > > > > > > > > >
> > > > > > > > > > > > Dr. Loris Bennett (Mr.)
> > > > > > > > > > >
> > > > > > > > > > > > Freie Universität Berlin
> > > > > > > > > > >
> > > > > > > > > > > > ZEDAT - Zentraleinrichtung für Datenverarbeitung /
> > > > Computer
> > > > > > > > > > > > Center
> > > > > > > > > > >
> > > > > > > > > > > > Compute & Media Service
> > > > > > > > > > >
> > > > > > > > > > > > Fabeckstr. 32, Room 221
> > > > > > > > > > >
> > > > > > > > > > > > D-14195 Berlin
> > > > > > > > > > >
> > > > > > > > > > > > Tel ++49 30 838 51024
> > > > > > > > > > >
> > > > > > > > > > > > Fax ++49 30 838 56721
> > > > > > > > > > >
> > > > > > > > > > > > Email 
> > > > > > > > > > > > 
> > > > > > > > > > >
> > > > > > > > > > > > Web www.zedat.fu-berlin.de
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > > Dr. Loris Bennett (Mr.)
> > > > > > > > > > Freie Universität Berlin
> > > > > > > > > > ZEDAT - Zentraleinrichtung für Datenverarbeitung /
> > Computer
> > > > > > Center
> > > > > > > > > > Compute & Media Service Fabeckstr. 32, Room 221
> > > > > > > > > > D-14195 Berlin
> > > > > > > > > > Tel ++49 30 838 51024
> > > > > > > > > > Fax ++49 30 838 56721
> > > > > > > > > > Email 
> > > > > > > > > > 
> > > > > > > > > > Web www.zedat.fu-berlin.de
> > > > > > > > >
> > > > > > > > --
> > > > > > > > Dr. Loris Bennett (Mr.)
> > > > > > > > Freie Universität Berlin
> > > > > > > > ZEDAT - Zentraleinrichtung für Datenverarbeitung / Computer
> > > > Center
> > > > > > > > Compute & Media Service Fabeckstr. 32, Room 221
> > > > > > > > D-14195 Berlin
> > > > > > > > Tel ++49 30 838 51024
> > > > > > > > Fax ++49 30 838 56721
> > > > > > > > Email 
> > > > > > > > 
> > > > > > > > Web www.zedat.fu-berlin.de
> > > > > > --
> > > > > > Dr. Loris Bennett (Mr.)
> > > > > > Freie Universität Berlin
> > > > > > ZEDAT - Zentraleinrichtung für Datenverarbeitung / Computer
> > Center
> > > > > > Compute & Media Service
> > > > > > Fabeckstr. 32, Room 221
> > > > > > D-14195 Berlin
> > > > > > Tel ++49 30 838 51024
> > > > > > Fax ++49 30 838 56721
> > > > > > Email 
> > > > > > 
> > > > > > Web www.zedat.fu-berlin.de
> > > > >
> > > > --
> > > > Dr. Loris Bennett (Mr.)
> > > > Freie Universität Berlin
> > > > ZEDAT - Zentraleinrichtung für Datenverarbeitung / Computer Center
> > > > Compute & Media Service
> > > > Fabeckstr. 32, Room 221
> > > > D-14195 Berlin
> > > > Tel ++49 30 838 51024
> > > > Fax ++49 30 838 56721
> > > > Email 
> > > > 
> > > > Web www.zedat.fu-berlin.de
> > >
> > --
> > Dr. Loris Bennett (Mr.)
> > Freie Universität Berlin
> > ZEDAT - Zentraleinrichtung für Datenverarbeitung / Computer Center
> > Compute & Media Service
> > Fabeckstr. 32, Room 221
> > D-14195 Berlin
> > Tel ++49 30 838 51024
> > Fax ++49 30 838 56721
> > Email 
> > 
> > Web www.zedat.fu-berlin.de
> 
-- 
Dr. Loris Bennett (Mr.)
Freie Universität Berlin
ZEDAT - Zentraleinrichtung für Datenverarbeitung / Computer Center
Compute & Media Service
Fabeckstr. 32, Room 221
D-14195 Berlin
Tel ++49 30 838 51024
Fax ++49 30 838 56721
Email 

Web www.zedat.fu-berlin.de