Skip to content

Enable GenericIndexed V2 for built-in(druid-io managed) complex columns. - #3987

Merged
fjy merged 2 commits into
apache:masterfrom
akashdw:enable_generic_indexed_v2
Mar 1, 2017
Merged

fjy merged 2 commits into
apache:masterfrom
akashdw:enable_generic_indexed_v2

Conversation

@akashdw

@akashdw akashdw commented Mar 1, 2017

Copy link
Copy Markdown
Contributor

This PR is to enable GenericIndexed V2 for complex columns.

@gianm

gianm commented Mar 1, 2017

Copy link
Copy Markdown
Contributor

This seems like something that should be 0.10.0 to complete the "large columns" stuff… @akashdw should we add it to the milestone?

@gianm gianm added this to the 0.10.0 milestone Mar 1, 2017
@akashdw

akashdw commented Mar 1, 2017 •

Copy link
Copy Markdown
Contributor Author

@gianm Yes, we need this to complete the large column stuff. I was going through the release notes and realized its disabled by default including druid-io managed complex metrics. I left a note in release notes on how to enable it. #3944 (comment)

@gianm

gianm commented Mar 1, 2017

Copy link
Copy Markdown
Contributor

Okay cool, I added this to the milestone.

@gianm gianm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@akashdw akashdw changed the title Enable GenericIndexed V2 for complex columns. Enable GenericIndexed V2 for built-in(druid-io managed) complex columns. Mar 1, 2017
@fjy

fjy commented Mar 1, 2017

Copy link
Copy Markdown
Contributor

👍

@fjy
fjy merged commit 91344cb into apache:master Mar 1, 2017
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.

3 participants