[Zope-Coders] Re: [Zope-Checkins] CVS: Zope/lib/python/Products/PluginIndexes/DateRangeIndex - DateRangeIndex.py:1.6.2.2

[Chris McDonough]

On Mon, 2003-11-03 at 20:03, Tres Seaver wrote:
> On Sun, 2003-11-02 at 06:59, Chris McDonough wrote:
> > Update of /cvs-repository/Zope/lib/python/Products/PluginIndexes/DateRangeIndex
> > In directory cvs.zope.org:/tmp/cvs-serv30007
> > 
> > Modified Files:
> >       Tag: Zope-2_7-branch
> > 	DateRangeIndex.py 
> > Log Message:
> > DateRangeIndexes blew up when attempting to store a date far in the future ( > Y2038 ). This is a workaround. The real fix would be to come up with a different conversion algorithm or to use OOBTrees to store datum.
> 
> I think a better fix here would be to raise ValueError if the Date is
> outside the "int-able" range;  otherwise, we end up moving the
> "surprise" point for the one where we have information (indexing) to the
> one where we don't (querying).

The problem with raising any error here is that it stops indexing from
happening.  Without really fixing the problem (using OOBTrees), we need
to prevent indexing from throwing errors (the other indexes do similar
stuff).  Maybe we could log the error?

- C