lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hoss Man (Commented) (JIRA)" <>
Subject [jira] [Commented] (SOLR-3218) Range faceting support for CurrencyField
Date Thu, 08 Mar 2012 22:57:59 GMT


Hoss Man commented on SOLR-3218:

bq. I believe start/end currency equality is enforced by MoneyType.compareTo which will throw
an exception when end is compared to the first (start+gap).

Ah ..ok.  and then ultimately start+gap is compared to end (even if hardend is false) so you'll
get a exception then.  ok fair enough.

bq. As far as enforcing currency equality being a good idea or not, it would make sense and
I would prefer if start/end/gap currencies didn't need to be equal. This patch doesn't allow
for that given the tradeoff of the utility of being able to use different currencies versus
the annoyance of keeping a handle open to an ExchangeRateProvider in the places we'd need

I'm not completley understanding, but i don't need to: If it's easier/simpler (for now) to
require that start/end/gap are all in the same currency that's fine -- we should just test/document
that clearly .. we can alwasy relax that restriction later if you think of a clean/easy way
to do it.

like i said before: it's probably silly to do it anyway, i just didn't understand if/what
the complication was
> 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
> 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