You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2013/07/24 14:43:52 UTC
[jira] [Commented] (LUCENE-5129) CategoryAssociationsContainer
should not allow null associations
[ https://issues.apache.org/jira/browse/LUCENE-5129?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13718324#comment-13718324 ]
ASF subversion and git services commented on LUCENE-5129:
---------------------------------------------------------
Commit 1506526 from [~shaie] in branch 'dev/trunk'
[ https://svn.apache.org/r1506526 ]
LUCENE-5129: CategoryAssociationsContainer should not allow null associations
> CategoryAssociationsContainer should not allow null associations
> ----------------------------------------------------------------
>
> Key: LUCENE-5129
> URL: https://issues.apache.org/jira/browse/LUCENE-5129
> Project: Lucene - Core
> Issue Type: Bug
> Components: modules/facet
> Reporter: Shai Erera
> Assignee: Shai Erera
> Fix For: 5.0, 4.5
>
> Attachments: LUCENE-5129.patch
>
>
> Currently you can set a null CategoryAssociation to a CategoryPath, but alas, it's just dropped by AssociationFacetFields. The history behind this break is that in the past, category associations were indexed twice, with and without the association value. In the move to 4.2 I guess with all the transitions the code went through, this behavior was nuked (as it doesn't make sense in general to index these categories twice), but AssociationFacetFields has a comment that these categories are added anyway -- this is of course wrong.
> Instead of supporting null category associations, which is not so trivial in AssociationFacetFields, I think that we should just prevent it (throw a hard exception). If a user wants to add a category such as Category/Computer Science=0.74 (i.e. with weight 0.74) and be able to aggregate Category/ with and without associations, he should either write a special FacetsAggregator which ignores the associated value, or add the category twice, once using FacetFields (counting) and second time with AssociationFacetFields.
> I'll post a patch soon.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org