Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to facet on a function #21

Open
wants to merge 1 commit into
base: bw_branch_7_4
Choose a base branch
from

Conversation

timatbw
Copy link

@timatbw timatbw commented May 18, 2018

No description provided.

@@ -103,42 +103,7 @@ public Object getMergedResult() {

sortBuckets();

long first = freq.offset;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This block of code has just been moved elsewhere

// sort : 'mystat desc'
// OR
// sort : { mystat : 'desc' }
protected void parseSort(FacetRequestSorted sortedFacet, Object sort) {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has just been moved from further down

@timatbw timatbw changed the base branch from bw_branch_7_2_1 to bw_branch_7_4 August 3, 2018 14:46
@timatbw
Copy link
Author

timatbw commented Aug 3, 2018

Rebased this one to 7.4 now. It can be tested with e.g. json.facet={ foo : { type: function, function: "mod(resource_id, 10)" } }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant