Skip to Content.
Sympa Menu

grouper-dev - RE: [grouper-dev] changelog implementation sketch

Subject: Grouper Developers Forum

List archive

RE: [grouper-dev] changelog implementation sketch


Chronological Thread 
  • From: Chris Hyzer <>
  • To: Tom Barton <>, Tom Zeller <>
  • Cc: "GW Brown, Information Systems and Computing" <>, Grouper Dev <>
  • Subject: RE: [grouper-dev] changelog implementation sketch
  • Date: Wed, 11 Jun 2008 02:09:30 -0400
  • Accept-language: en-US
  • Acceptlanguage: en-US

Some thoughts:

1. From doing the POC in hooks, I noticed that not all info is available at
time of hibernate call. E.g. when adding group A to group B, all the members
of A are given a membership in group B. But the subjectIds aren't even
known, it just uses the member uuid's. Similarly, another example is if a
subject is added to group A, and group A is a member of group B, then a
membership record is added for the subject in group B. But the group name is
not known, only the UUID. So if we are going to have a friendly
representation of change, then it would result in more DB select queries just
to get that info.

2. There are lots of queries going on for each operation. E.g. to add a
member to a group, it is multiple queries (e.g. above). And if each of those
queries results in an audit query, and each audit query goes to the DB for a
sequential ID, and potentially reads more info, I don't see how we aren't
going to have a slow down by a factor of 1.5-3 (depending on the operation).
But of course we can test this.

3. I think packing data in a field (e.g. the LDIF design) to be extrapolated
later by systems frequently sounds good at first, but later it might be
wished to be relational. If we are storing all data about what has changed,
how about we:

a. Make a table for each table we want to track
b. Insert a "shadow" record in the appropriate table for each DB operation
c. This would be done in java, but for people who want high performance, they
could turn off the java part and just use triggers (this should make
significantly faster)

Just brainstorming here... :)

Kind regards,
Chris


> -----Original Message-----
> From: Tom Barton
> [mailto:]
> Sent: Tuesday, June 10, 2008 10:50 PM
> To: Tom Zeller
> Cc: GW Brown, Information Systems and Computing; Grouper Dev
> Subject: Re: [grouper-dev] changelog implementation sketch
>
> For audit, the first order of business is to preserve the information,
> and for that purpose (alone) we don't need to store the info in both a
> group-centric and a member-centric way.
>
> Actually answering point-in-time questions should be of rather low
> frequency compared to using the info for incremental change
> notification, and it's only the latter that has a need for low latency.
>
> So I think we can pick the single representation (group-centric or
> member-centric) best suited to sensibly representing changes. And since
> not all changes to group info are membership related, it looks like
> group-centric is the only single one that meets the need.
>
> Tom
>
> Tom Zeller wrote:
> > On Tue, Jun 10, 2008 at 10:12 AM, GW Brown, wrote:
> >
> >
> > --On 10 June 2008 09:17 -0500 Tom Zeller
> > <
> >
> > <mailto:>>
> > wrote:
> >
> >
> > Representing changes from a group-centric point of view via
> LDIF
> > seems
> > straightforward, but what about member-centrically ? e.g. how
> > someone's
> > group memberships have changed.
> >
> > What is the context here? Are we back to the subject id changing
> or
> > is this to make it easier to query from a member point of view
> i.e.
> > show me all the memberships for 'x' on a given date?
> >
> >
> > The latter, make it easier/possible to query changes from a member
> > point of view.
> >
> >
> >
> > If the actual change definitions - the LDIF or whatever
> format
> > we use - for a group can be queried using 'like' we could
> have
> > an inefficient means of finding relevant changes which would
> > allow us to compute a subject's memberships at a given point
> in
> > time.
> >
> >
> > I agree: querying LDIF via SQL seems inefficient, perhaps even
> improper.
> >



Archive powered by MHonArc 2.6.16.

Top of Page