-
Notifications
You must be signed in to change notification settings - Fork 368
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
Feature Store API: Store all/most as search templates #32
Comments
@epugh, this ticket was created from the "Elasticsearch LTR version 1.0" roadmap in GoogleDocs, so maybe @softwaredoug or @nomoa can elaborate more. https://docs.google.com/document/d/1_DWPmLi9oDem3QWxQAgKbqZ5F9oo0XXyHicsXmbWFDQ/edit# |
nomoa is taking a well deserved vacation, will be out until basically the beginning of next month. I'm not 100% sure what this is, but the feature store has 3 ways of storing features: A plain un-parameterized elasticsearch query, a mustache template, or a lucene expression (taking the features from the other 2 as parameters). So, basically everything is a template unless you want to use something else. It seems "done" ? |
Yep, sounds like it’s done barring some documentation covering those three approaches!
… On Aug 8, 2017, at 5:19 PM, ebernhardson ***@***.***> wrote:
nomoa is taking a well deserved vacation, will be out until basically the beginning of next month. I'm not 100% sure what this is, but the feature store has 3 ways of storing features: A plain un-parameterized elasticsearch query, a mustache template, or a lucene expression (taking the features from the other 2 as parameters).
So, basically everything is a template unless you want to use something else. It seems "done" ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#32 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABXe7cBOs-VEsYddoE9AoNnUJ7ZXD8-ks5sWNDfgaJpZM4NPvD_>.
|
I think this is done |
May not be possible. Let's keep track of the roadblocks if that's the case so that we might revisit later.
The text was updated successfully, but these errors were encountered: