Yeah, I agree with you, Chris. I think these setters should be removed.
On Wed, Oct 9, 2013 at 1:33 PM, Daan Hoogland <daan.hoogl...@gmail.com>wrote: > Chris, > > Since I see no objections, why don't you test your idea and submit a patch? > > regards, > Daan > > On Fri, Oct 4, 2013 at 7:29 PM, SuichII, Christopher > <chris.su...@netapp.com> wrote: > > *JoinVOs are used to store entries from MySQL views, which are not > editable. I think removing setters from the *JoinVOs may help avoid some > potential confusion as setters seem to imply that the fields are editable, > which they really aren't. > > > > I started looking around and it looks like most setters in *JoinVOs > aren't actually used since the creation of *VOs is handled by java > reflection. Please let me know if this is not the case or if I'm > misunderstanding the way the MySQL views work. > > > > -Chris > > -- > > Chris Suich > > chris.su...@netapp.com > > NetApp Software Engineer > > Data Center Platforms – Cloud Solutions > > Citrix, Cisco & Red Hat > > > -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e: mike.tutkow...@solidfire.com o: 303.746.7302 Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play> *™*
