commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz" <phil.ste...@gmail.com>
Subject Re: [math] understanding BigMatrixImpl.TOO_SMALL field
Date Mon, 14 Jan 2008 05:35:14 GMT
On Jan 12, 2008 4:49 PM, Phil Steitz <phil.steitz@gmail.com> wrote:
> On Jan 12, 2008 3:45 PM, Rahul Akolkar <rahul.akolkar@gmail.com> wrote:
> > On 1/12/08, Luc Maisonobe <Luc.Maisonobe@free.fr> wrote:
> > > One comment about this change: this would break compatibility with
> > > version 1.1 and the clirr plugin flags this as an error.
> > <snip/>
> >
> > That was my initial reaction as well when I read your first note
> > (below). In general, we try to avoid incompatible changes (there are
> > specific cases where we can attempt to push for exceptions, such as
> > blatant bugs and/or v0.x releases).
> >
> > About the clirr errors you mention in the following paragraph, I
> > suspect they will come up for discussion at the next release unless
> > they are addressed or its a major release etc.
> >
> Yes.  We need to fix these things somehow.  The decision to make these
> fields protected in 1.0 was unfortunate, but we need to find a way to
> maintain compatibility while we deprecate things.  Sorry, I think this
> was my mistake.  I will look into how to fix the problems in the stats
> package.

I remember now what is going on here.  The protected fields "removed"
from DescriptiveStatisticsImpl, SummaryStatisticsImpl were moved to
their superclasses, so are still available to them and their
subclasses.  I think this is OK, just should be commented in release
notes.

Phil

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message