Skip to content

Conversation

@cbuescher
Copy link
Member

With #34811 the API for stopping rollup jobs got two new url parameters
"wait_for_completion" and "timeout". This change adds these to the HLRC APIs as
well.

Relates to #34811

With elastic#34811 the API for stopping rollup jobs got two new url parameters
"wait_for_completion" and "timeout". This change adds these to the HLRC APIs as
well.

Relates to elastic#34811
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@cbuescher cbuescher changed the title HLRC: Add parameters to stopRollupJon API HLRC: Add parameters to stopRollupJob API Nov 14, 2018
Copy link
Contributor

@polyfractal polyfractal left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@cbuescher
Copy link
Member Author

@polyfractal thanks for the review, I just added another commit extending the HL client docs with the two new parameters, in case you want to take another last look at those. Otherwise I'll wait for green and merge after that.

@polyfractal
Copy link
Contributor

Ah good catch, I forgot about the docs for HLRC. ++ :)

@cbuescher cbuescher merged commit bc5b1af into elastic:master Nov 15, 2018
cbuescher pushed a commit that referenced this pull request Nov 15, 2018
With #34811 the API for stopping rollup jobs got two new url parameters
"wait_for_completion" and "timeout". This change adds these to the HLRC APIs as
well.

Relates to #34811
@tomcallahan tomcallahan removed the :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data label Dec 18, 2018
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.

5 participants