lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christine Poerschke (JIRA)" <>
Subject [jira] [Commented] (SOLR-13049) make contrib/ltr Feature.defaultValue configurable
Date Fri, 03 May 2019 18:29:00 GMT


Christine Poerschke commented on SOLR-13049:

Finally returning to this ticket ... thanks [~slivotov] for your input!

Attached patch passes {{ant precommit}} (including {{ant check-forbidden-apis}}) on the current
master branch after four straightforward
-        defaultValue = new Float(0.5f);
+        defaultValue = Float.valueOf(0.5f);
style replacements. {{cd solr/contrib/ltr ; ant beast -Dbeast.iters=10}} also passed.

If there are no further comments, questions, concerns, etc. then I'll aim to commit this patch
next week.

> make contrib/ltr Feature.defaultValue configurable
> --------------------------------------------------
>                 Key: SOLR-13049
>                 URL:
>             Project: Solr
>          Issue Type: New Feature
>          Components: contrib - LTR
>            Reporter: Stanislav Livotov
>            Priority: Major
>         Attachments: SOLR-13049.patch, SOLR-13049.patch
> [~slivotov] wrote in SOLR-12697:
> {quote}
> I had also done a couple of additional code changes:
> 1. fixed small issue with defaultValue(previously it was impossible to set it from feature.json,
and the tests were written where Feature was created manually, and not by parsing json). Tests
are added which are validating defaultValue from schema field configuration and from a feature
default value.
> {quote}
> (Please see
for more context.)

This message was sent by Atlassian JIRA

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

View raw message