[ 
https://issues.apache.org/jira/browse/SOLR-2772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13154551#comment-13154551
 ] 

Hoss Man commented on SOLR-2772:
--------------------------------

I guess i don't understand what exactly you mean by "original". 

The *original* circa Solr 1.1 hand rolled code for dealing with dates had all 
sorts of bugs, fixed in various ways over the years -- this particular patch 
fixed two bugs in DateField (related to years 0001-1000 and millis on years 
before the epoch) and corrected the fact that TextResponseWriter was 
erroneously not using the well tested formatting code in DateField -- so it was 
still susceptible to some of those old bugs.  (do i have the full list of those 
bugs handy: no, but if you're interested you can search Jira for them)

If you want to re-write this to be hand rolled go ahead, but please do so in 
the DateField code (and don't just revert the changes to TextResponseWriter) 
since tests directly against DateField are where most of the Unit testing of 
parsing/formatting esoteric dates live -- if you keep all those tests happy 
(w/o writting yet another new set of parse/format methods) then i'll happily 
welcome whatever high-performance hand rolled solution you have.
                
> DateField parsing/formatting issues of years 0001-1000
> ------------------------------------------------------
>
>                 Key: SOLR-2772
>                 URL: https://issues.apache.org/jira/browse/SOLR-2772
>             Project: Solr
>          Issue Type: Sub-task
>            Reporter: Hoss Man
>            Assignee: Hoss Man
>             Fix For: 3.5, 4.0
>
>
> see parent task for background

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to