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

derivequery: pass search filter to query #152

Merged
merged 3 commits into from
Jun 21, 2013
Merged

derivequery: pass search filter to query #152

merged 3 commits into from
Jun 21, 2013

Conversation

nikicat
Copy link

@nikicat nikicat commented Jun 10, 2013

This patch corrects derived queries.

@rashidkpc
Copy link
Contributor

I'd like to see this be configurable perhaps with 3 modes: terms alone, AND, or OR

@nikicat
Copy link
Author

nikicat commented Jun 11, 2013

Hmm, terms alone, AND is ok, but what's the case for OR?

@rashidkpc
Copy link
Contributor

A case for OR would be something like this. Say I have this set of logs, with fields foo and bar, each of which get a unique ID used for tracking a session.
foo=1
foo=1 bar=2
bar=2

Now say I want to track the progress of the session that started as foo=1. All I know is foo=1 and it will eventually get a bar value, and eventually lose its foo value. OR would allow me to derive a list of values for bar based on foo, so I would end up with:

foo:1 OR bar:2

Which would match all of the above events

@nikicat
Copy link
Author

nikicat commented Jun 12, 2013

I've updated the patch

rashidkpc pushed a commit that referenced this pull request Jun 21, 2013
derivequery: pass search filter to query
@rashidkpc rashidkpc merged commit 41ea74f into elastic:master Jun 21, 2013
spalger pushed a commit that referenced this pull request Oct 6, 2014
spalger pushed a commit that referenced this pull request Oct 6, 2014
harper-carroll referenced this pull request in harper-carroll/kibana Jul 11, 2016
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.

2 participants