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
ES backended filer support kv ops. #1449
Conversation
Is there any way to tell Elastic not to index the content? https://www.elastic.co/guide/en/elasticsearch/reference/current/enabled.html#enabled |
separate into an |
Index(indexKV). | ||
Type(indexType). | ||
Id(id). | ||
Do(context.Background()) |
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.
there is already a ctx
parameter. No need to use context.Background()
. Same for other places.
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.
it can be without indexing if totally no need to query.
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.
Only need to fetch the value by the key.
deleteResult, err := store.client.Delete(). | ||
Index(indexKV). | ||
Type(indexType). | ||
Id(id). |
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 not use key
here?
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.
will cause problem if key contains '/'.
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.
no need to worry about that.
2.disable query for kv in es index.
No description provided.