-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Star tree] Support for keyword changes as part of star tree mapper #16103
base: main
Are you sure you want to change the base?
[Star tree] Support for keyword changes as part of star tree mapper #16103
Conversation
Signed-off-by: Bharathwaj G <[email protected]>
Signed-off-by: Bharathwaj G <[email protected]>
…timestamp-support
Signed-off-by: Bharathwaj G <[email protected]>
…timestamp-support
…timestamp-support
…timestamp-support
Signed-off-by: Bharathwaj G <[email protected]>
…timestamp-support
…timestamp-support
Signed-off-by: Bharathwaj G <[email protected]>
…timestamp-support
❌ Gradle check result for 249fa53: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Signed-off-by: Bharathwaj G <[email protected]>
249fa53
to
49dd2f5
Compare
❌ Gradle check result for 49dd2f5: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
if (hasDocValues) { | ||
return "numeric"; | ||
} else if (isKeyword) { | ||
return "keyword"; | ||
return "ip"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why are we returning "ip" in the above two ifs?
return DocValues.emptySortedNumeric(); | ||
} | ||
}); | ||
if(isSortedSetField(compositeField)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we have a switch? for when we support more field types.
Description
This PR supports keyword fields as part of the star tree mapping.
This is built on top of date mapping PR.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.