Raphael Luta wrote:
> +1 for list(locator) being in Profiler. It's definitely
> Profiler's job to do
> that, although I would amend the interface to
>
> public Iterator list( RunData data, ProfileLocator locator )
> or
> public Iterator list( User user, ProfileLocator locator )
> or
> add a get/setUser() in ProfileLocator to put in the RunData User object.
>
> Why ? Imagine I want to write an alternate adaptive Profiler that stores
> persistent information in the User profile... I need to be able to access
> a pointer to the User.
> ... All in all, I think the third option is the best, it replaces
> the string
> user parameter that was there before and gives a lot more power to the
> profiling interface and enables a trivial implementation of a
> DB-backed PSML
> implementation (just store PSML as BLOB using setPerm())
>
> What do you think ?

Good point, I will update the ProfilerLocator to have a User object instead
of a String.
Do you hold the same argument for Roles?




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to