Skip to content

add rollup index check before doing rollup #79596

Closed
weizijun wants to merge 1 commit intoelastic:masterfrom
weizijun:rollupv2-check-rollup-index
Closed

add rollup index check before doing rollup #79596
weizijun wants to merge 1 commit intoelastic:masterfrom
weizijun:rollupv2-check-rollup-index

Conversation

@weizijun
Copy link
Copy Markdown
Contributor

As rollup is a heavy action. I prefer check rollup index exists before doing rollup.

@elasticsearchmachine elasticsearchmachine added v8.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Oct 21, 2021
@weizijun
Copy link
Copy Markdown
Contributor Author

@csoulios @jimczi can you help to review this PR?

@jtibshirani jtibshirani added the :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data label Oct 28, 2021
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 28, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@arteam arteam added v8.1.0 and removed v8.0.0 labels Jan 12, 2022
@mark-vieira mark-vieira added v8.2.0 and removed v8.1.0 labels Feb 2, 2022
@csoulios csoulios self-assigned this Jun 1, 2022
@csoulios
Copy link
Copy Markdown
Contributor

csoulios commented Jun 1, 2022

This PR overlaps with the work done in PR #85708

@weizijun thank you for this contribution.

@csoulios csoulios closed this Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external-contributor Pull request authored by a developer outside the Elasticsearch team :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants