[jira] Created: (LUCENE-788) contrib/benchmark assumes Locale.US for parsing dates in Reuters collection

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (LUCENE-788) contrib/benchmark assumes Locale.US for parsing dates in Reuters collection

JIRA jira@apache.org
contrib/benchmark assumes Locale.US for parsing dates in Reuters collection
---------------------------------------------------------------------------

                 Key: LUCENE-788
                 URL: https://issues.apache.org/jira/browse/LUCENE-788
             Project: Lucene - Java
          Issue Type: Bug
    Affects Versions: 2.1
            Reporter: Doron Cohen
         Assigned To: Doron Cohen
            Priority: Minor
             Fix For: 2.1


SimpleDateFormat used for parsing dates in Reuters documents is instantiated without specifying a locale. So it is using the default locale. If that happens to be US, it will work. But for another locale a parse exception is likely.

Affects both StandardBenchmarker and ReutersDocMaker.

Fix is trivial - specify Locale.US for SimpleDateFormat's constructor.


--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (LUCENE-788) contrib/benchmark assumes Locale.US for parsing dates in Reuters collection

JIRA jira@apache.org

     [ https://issues.apache.org/jira/browse/LUCENE-788?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Doron Cohen updated LUCENE-788:
-------------------------------

    Attachment: 788_benchmark_parseDate_locale_Jan_27.patch

Locale.US passed to SimpleDateFormat constructor.

> contrib/benchmark assumes Locale.US for parsing dates in Reuters collection
> ---------------------------------------------------------------------------
>
>                 Key: LUCENE-788
>                 URL: https://issues.apache.org/jira/browse/LUCENE-788
>             Project: Lucene - Java
>          Issue Type: Bug
>    Affects Versions: 2.1
>            Reporter: Doron Cohen
>         Assigned To: Doron Cohen
>            Priority: Minor
>             Fix For: 2.1
>
>         Attachments: 788_benchmark_parseDate_locale_Jan_27.patch
>
>
> SimpleDateFormat used for parsing dates in Reuters documents is instantiated without specifying a locale. So it is using the default locale. If that happens to be US, it will work. But for another locale a parse exception is likely.
> Affects both StandardBenchmarker and ReutersDocMaker.
> Fix is trivial - specify Locale.US for SimpleDateFormat's constructor.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (LUCENE-788) contrib/benchmark assumes Locale.US for parsing dates in Reuters collection

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/jira/browse/LUCENE-788?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Doron Cohen updated LUCENE-788:
-------------------------------

    Lucene Fields: [New, Patch Available]  (was: [New])

> contrib/benchmark assumes Locale.US for parsing dates in Reuters collection
> ---------------------------------------------------------------------------
>
>                 Key: LUCENE-788
>                 URL: https://issues.apache.org/jira/browse/LUCENE-788
>             Project: Lucene - Java
>          Issue Type: Bug
>    Affects Versions: 2.1
>            Reporter: Doron Cohen
>         Assigned To: Doron Cohen
>            Priority: Minor
>             Fix For: 2.1
>
>         Attachments: 788_benchmark_parseDate_locale_Jan_27.patch
>
>
> SimpleDateFormat used for parsing dates in Reuters documents is instantiated without specifying a locale. So it is using the default locale. If that happens to be US, it will work. But for another locale a parse exception is likely.
> Affects both StandardBenchmarker and ReutersDocMaker.
> Fix is trivial - specify Locale.US for SimpleDateFormat's constructor.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[jira] Closed: (LUCENE-788) contrib/benchmark assumes Locale.US for parsing dates in Reuters collection

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/jira/browse/LUCENE-788?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Grant Ingersoll closed LUCENE-788.
----------------------------------

       Resolution: Fixed
    Lucene Fields:   (was: [Patch Available, New])

Fixed with LUCENE-790

> contrib/benchmark assumes Locale.US for parsing dates in Reuters collection
> ---------------------------------------------------------------------------
>
>                 Key: LUCENE-788
>                 URL: https://issues.apache.org/jira/browse/LUCENE-788
>             Project: Lucene - Java
>          Issue Type: Bug
>    Affects Versions: 2.1
>            Reporter: Doron Cohen
>         Assigned To: Doron Cohen
>            Priority: Minor
>             Fix For: 2.1
>
>         Attachments: 788_benchmark_parseDate_locale_Jan_27.patch
>
>
> SimpleDateFormat used for parsing dates in Reuters documents is instantiated without specifying a locale. So it is using the default locale. If that happens to be US, it will work. But for another locale a parse exception is likely.
> Affects both StandardBenchmarker and ReutersDocMaker.
> Fix is trivial - specify Locale.US for SimpleDateFormat's constructor.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]