Hi Varun, The changes look fine to me. However, the fix should also include doc changes to the "data service developers' guide". Check this: http://docs.sun.com/app/docs/doc/820-4685/scds-property-functions-3ha?l=en&a=view "Extension Property-Specific Functions" should include Child_mon_level now. Of course, the guide fails to mention some other properties as well. You can submit a follow-up doc bug to track this.
Thanks, Swathi On 04/07/09 05:03, Martin Rattner wrote: > I am re-sending this because some ha-clusters-discuss members inside > of Sun did not receive it by email when it was originally posted. > > Any code reviews would be appreciated. > > -------- Original Message -------- > Posted: Mar 25, 2009 2:20 AM > To: Communities ? ha-clusters ? discuss > > Hi, > > I am Varun Rao,a student of Amrita Vishwa Vidyapeetham,Kerala. > > I recently fixed bug in the sun cluster (Bug ID 6603524). > (http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6603524) > > My sponsor is Mr. Martin Rattner. > The webrev is put up at > http://cr.opensolaris.org/~mrattner/webrev_6603524/ > > I kindly request all of you to review this code change and let me know > if any further changes can be incorporated. > > Regards, > Varun Rao > Amrita Vishwa Vidyapeetham,Kerala > _______________________________________________ > ha-clusters-discuss mailing list > ha-clusters-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/ha-clusters-discuss >
