[jira] [Updated] (SOLR-2255) local params are not parsed in facet.pivot parameter
[
https://issues.apache.org/jira/browse/SOLR-2255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
David Smiley updated SOLR-2255:
---
Attachment: SOLR-2255_local-param_support_for_pivot_faceting.patch
This is a small update to the patch to update it for a recent change. FYI this
works on 4x too.
Any comments from the likes of Yonik, Hoss, Ryan, or Erik (those involved in
the underlying code)?
local params are not parsed in facet.pivot parameter
Key: SOLR-2255
URL: https://issues.apache.org/jira/browse/SOLR-2255
Project: Solr
Issue Type: New Feature
Affects Versions: 4.0-ALPHA
Reporter: Julien Lirochon
Assignee: David Smiley
Attachments: SOLR-2255_local-param_support_for_pivot_faceting.patch,
SOLR-2255_local-param_support_for_pivot_faceting.patch
...facet=truefacet.pivot={!ex=category}category_id,subcategory_idfq={!tag=category}category_id=42
generates the following error : undefined field {!ex=category}category_id
If you filter on subcategory_id, the facet.pivot result will contain only
results from this subcategory. It's a loss of function since you can't alter
this behavior with local params.
--
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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Updated] (SOLR-2255) local params are not parsed in facet.pivot parameter
[
https://issues.apache.org/jira/browse/SOLR-2255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
David Smiley updated SOLR-2255:
---
Issue Type: New Feature (was: Bug)
local params are not parsed in facet.pivot parameter
Key: SOLR-2255
URL: https://issues.apache.org/jira/browse/SOLR-2255
Project: Solr
Issue Type: New Feature
Affects Versions: 4.0-ALPHA
Reporter: Julien Lirochon
Assignee: David Smiley
...facet=truefacet.pivot={!ex=category}category_id,subcategory_idfq={!tag=category}category_id=42
generates the following error : undefined field {!ex=category}category_id
If you filter on subcategory_id, the facet.pivot result will contain only
results from this subcategory. It's a loss of function since you can't alter
this behavior with local params.
--
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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Updated] (SOLR-2255) local params are not parsed in facet.pivot parameter
[
https://issues.apache.org/jira/browse/SOLR-2255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
David Smiley updated SOLR-2255:
---
Attachment: SOLR-2255_local-param_support_for_pivot_faceting.patch
Attached is a patch developed against trunk. It'll probably apply cleanly
against 4x but I haven't tried yet.
Adding this feature was awkward because of the nature of the PivotFacetHelper
and SimpleFacet classes. The core of the problem is that PivotFacetHelper
needs to invoke SimpleFacet.parseParams() somehow but that method is package
access. And the variables it sets are package access as well. It is invoked
indirectly via getFacetFieldCounts() but PivotFacetHelper sidesteps that method
and calls getTermCounts() so that the pivot code can split on commas.
PivotFacetHelper was oddly (IMO) declared as a thread-safe class (it has no
state) and there is a shared instance of it in FacetComponent but its not clear
why; it's a trivial class. It has an extension hook getFacetImplementation()
with a default impl that simply instantiates a new SimpleFacet instance. Yet I
found no case of this method being overridden, and it doesn't seem like an
extension point worth keeping. It's not clear that the SimpleFacet behemoth
was designed to be substituted/extended.
Whatever I did would be somewhat disruptive to the relationship between these
classes so I took a stab at it any way. In my patch, PivotFacetHelper extends
SimpleFacet and is instantiated similarly in FacetComponent. I removed the
dubious getFacetImplementation() method in favor of simply referencing the
current instance this. This also made it possible to remove some parameters
(which was getting a little out of control) since they are inherited fields
now. I changed package access to protected to parseParams() and the fields it
sets. I also decided to do a refactor rename there of docs to docsOrig and
base to docs so that it's more clear what's going on there. It's not clear
to me if references in this class to docsOrig is an undiscovered bug or not.
The test demonstrates a filter query exclusion as well as key to add the data
to the response using a different name. I noticed there is a threads
performance tuning option that should now work too.
local params are not parsed in facet.pivot parameter
Key: SOLR-2255
URL: https://issues.apache.org/jira/browse/SOLR-2255
Project: Solr
Issue Type: New Feature
Affects Versions: 4.0-ALPHA
Reporter: Julien Lirochon
Assignee: David Smiley
Attachments: SOLR-2255_local-param_support_for_pivot_faceting.patch
...facet=truefacet.pivot={!ex=category}category_id,subcategory_idfq={!tag=category}category_id=42
generates the following error : undefined field {!ex=category}category_id
If you filter on subcategory_id, the facet.pivot result will contain only
results from this subcategory. It's a loss of function since you can't alter
this behavior with local params.
--
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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
