Skip to content

Composite chunk store#877

Closed
tomwilkie wants to merge 9 commits intocortexproject:masterfrom
grafana:composite-chunk-store
Closed

Composite chunk store#877
tomwilkie wants to merge 9 commits intocortexproject:masterfrom
grafana:composite-chunk-store

Conversation

@tomwilkie
Copy link
Contributor

@tomwilkie tomwilkie commented Jul 12, 2018

(includes #870 and #713)

This will allow us to vary the store implementation over time, and not just the schema. This will unblock the new bigtable storage adapter (using columns instead of rows, see #753), and allow us to more easily implement the iterative intersections and indexing of series instead of chunks (see #875). IMO this is a key step in unlocking better query performance.

tomwilkie added 9 commits July 6, 2018 16:24
Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
- Refactor chunk store to return chunks.
- Refactor querier package to use new Prometheus 2.0 interfaces.
- Have separate querier for ingesters, chunk store and metadata.
- Make remote read handler take a Queryable.

Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
… PromQL query.

Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
…and the upstream heap-based merging code.

Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
…ate NaN values

Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
This will allow us to vary the store implementation over time, and not just the schema.
This will unblock the new bigtable storage adapter (using columns instead of rows), and allow us to more easily implement the iterative intersections and indexing of series instead of chunks.

Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>
@tomwilkie tomwilkie force-pushed the composite-chunk-store branch from ee80739 to bf85827 Compare July 12, 2018 11:43
@tomwilkie tomwilkie changed the title [WIP] Composite chunk store Composite chunk store Jul 12, 2018
@tomwilkie
Copy link
Contributor Author

Think I'll just include this as part of #875.

@tomwilkie tomwilkie closed this Jul 13, 2018
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.

1 participant