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

Support projection post operator #3285

Closed
wants to merge 1 commit into from

Conversation

navis
Copy link
Contributor

@navis navis commented Jul 26, 2016

To retain only required columns in result. We can make columns via post-aggregator, etc. but cannot remove column which is not needed or should not be returned.

@navis
Copy link
Contributor Author

navis commented Jul 29, 2016

example:

"context": {
   "postProcessing" : {
      "type": "projection",
      "columns": ["column", "list", "you", "want", "to", "retain"]
   }
}

@gianm
Copy link
Contributor

gianm commented Jul 29, 2016

My feeling is that this is a neat feature but would better belong in the ToolChests (probably in postMergeQueryDecoration) than in a post processing operator. That way it would be extensible for custom queries.

@gianm gianm added the Feature label Jul 29, 2016
@navis
Copy link
Contributor Author

navis commented Jul 29, 2016

@gianm Thought about that but for that we need add new fields (something like "retainColumns") in query objects. would it be better?

@gianm
Copy link
Contributor

gianm commented Jul 29, 2016

IMO it wouldn't be too bad to add an optional "projection" field to any query that wants to support projections. welcome opinions from others though.

@navis
Copy link
Contributor Author

navis commented Aug 24, 2016

@gianm addressed comment. added outputColumns to timeseries/groupby/topn/select queries.

@stale
Copy link

stale bot commented Feb 28, 2019

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@druid.apache.org list. Thank you for your contributions.

@stale stale bot added the stale label Feb 28, 2019
@stale
Copy link

stale bot commented Mar 7, 2019

This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

@stale stale bot closed this Mar 7, 2019
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Jan 10, 2020
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants