+1. Thanks, Sangjin
On Thu, May 12, 2016 at 10:05 AM, Ravi Prakash <ravihad...@gmail.com> wrote: > Thanks sounds reasonable Colin. +1 to not using toString() as an API > > On Thu, May 12, 2016 at 9:57 AM, Chris Nauroth <cnaur...@hortonworks.com> > wrote: > > > I'm in favor of making a statement in the compatibility guidelines that > > there is no guarantee of stability or backwards-compatibility for > > toString() output. If a proposed patch introduces new dependencies on a > > stable toString() output, such as for UI display or object serialization, > > then I'd consider -1'ing it and instead asking that the logic move to a > > different method that can provide that guarantee, i.e. toStableString(). > > There are further comments justifying this on HADOOP-13028 and HDFS-9732. > > > > --Chris Nauroth > > > > > > > > > > On 5/12/16, 9:32 AM, "Colin McCabe" <cmcc...@apache.org> wrote: > > > > >Hi all, > > > > > >Recently a discussion came up on HADOOP-13028 about the wisdom of > > >overloading S3AInputStream#toString to output statistics information. > > >It's a difficult judgement for me to make, since I'm not aware of any > > >compatibility guidelines for InputStream#toString. Do we have > > >compatibility guidelines for toString functions? > > > > > >It seems like the output of toString functions is usually used as a > > >debugging aid, rather than as a stable format suitable for UI display or > > >object serialization. Clearly, there are a few cases where we might > > >want to specifically declare that a toString method is a stable API. > > >However, I think if we attempt to treat the toString output of all > > >public classes as stable, we will have greatly increased the API > > >surface. Should we formalize this and declare that toString functions > > >are @Unstable, Evolving unless declared otherwise? > > > > > >best, > > >Colin > > > > > >--------------------------------------------------------------------- > > >To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org > > >For additional commands, e-mail: common-dev-h...@hadoop.apache.org > > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org > > For additional commands, e-mail: common-dev-h...@hadoop.apache.org > > > > >
