lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Morrison (Commented) (JIRA)" <>
Subject [jira] [Commented] (SOLR-3218) Range faceting support for CurrencyField
Date Mon, 12 Mar 2012 01:42:39 GMT


Andrew Morrison commented on SOLR-3218:

The latest patch is updated to account for the CurrencyField name.

I updated CurrencyValue.toString() to return "3.14,USD" for $3.14 rather than "314,USD". My
feeling is that it's more straight forward to return strings that look like the values that
were passed in to parse(). If we don't want toString to act as such, I can move that logic
to CurrencyRangeEndpointCalculator.formatValue(). Also of interest, I'm injecting a '.' into
the toString value of the long amount rather than using any existing floating point number
formatters which might use a ',' rather than a '.' causing issues with the existing comma
delimiter between the amount and currencyCode.


It was easy to allow the gap to be in any currency for which we have a conversion, so we're
now allowing that. Start and end must still be of the same currency. I worry that relaxing
the restriction on the gap may just be confusing without adding any real value. We may want
to consider forcing gap to be the same as start and end so that things are more conceptually
straight forward.
> Range faceting support for CurrencyField
> ----------------------------------------
>                 Key: SOLR-3218
>                 URL:
>             Project: Solr
>          Issue Type: Improvement
>          Components: Schema and Analysis
>            Reporter: Jan H√łydahl
>             Fix For: 4.0
>         Attachments: SOLR-3218-1.patch, SOLR-3218-2.patch, SOLR-3218.patch
> Spinoff from SOLR-2202. Need to add range faceting capabilities for CurrencyField

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message