Skip to content

[CCR] Refactor stats APIs #34912

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

Merged
merged 3 commits into from
Oct 29, 2018

Conversation

martijnvg
Copy link
Member

  • Changed the auto follow stats to also include follow stats.
  • Renamed the auto follow stats api to stats api and changed its url path
    from /_ccr/auto_follow/stats /_ccr/stats.
  • Removed /_ccr/stats url path for the follow stats api, which makes
    the index parameter a required parameter.

* Changed the auto follow stats to also include follow stats.
* Renamed the auto follow stats api to stats api and changed its url path
  from `/_ccr/auto_follow/stats` `/_ccr/stats`.
* Removed `/_ccr/stats` url path for the follow stats api, which makes
  the index parameter a required parameter.
@martijnvg martijnvg added >non-issue :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features labels Oct 26, 2018
@martijnvg martijnvg requested a review from dnhatn October 26, 2018 14:48
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

* master:
  Introduce cross-cluster replication API docs (elastic#34726)
  Responses can use Writeable.Reader interface (elastic#34655)
  SQL: Provide null-safe scripts for Not and Neg (elastic#34877)
  Fix put/resume follow request parsing (elastic#34913)
  Fix line length for org.elasticsearch.common.* files (elastic#34888)
  [ML] Extract common native process base class (elastic#34856)
  Refactor children aggregator into a generic ParentJoinAggregator (elastic#34845)
  [Style] Fix line lengths in action.admin.indices (elastic#34890)
  HLRC - add support for source exists API (elastic#34519)
Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@martijnvg martijnvg merged commit 1801518 into elastic:master Oct 29, 2018
martijnvg added a commit that referenced this pull request Oct 29, 2018
* Changed the auto follow stats to also include follow stats.
* Renamed the auto follow stats api to stats api and changed its url path
  from `/_ccr/auto_follow/stats` `/_ccr/stats`.
* Removed `/_ccr/stats` url path for the follow stats api, which makes
  the index parameter a required parameter.
* Fixed docs.
martijnvg added a commit that referenced this pull request Oct 29, 2018
* Changed the auto follow stats to also include follow stats.
* Renamed the auto follow stats api to stats api and changed its url path
  from `/_ccr/auto_follow/stats` `/_ccr/stats`.
* Removed `/_ccr/stats` url path for the follow stats api, which makes
  the index parameter a required parameter.
* Fixed docs.
dliappis added a commit to dliappis/rally that referenced this pull request Oct 29, 2018
The ccr-stats API endpoint has been updated in [1]; update the
telemetry device and tests to align with the changes introduced.

[1] elastic/elasticsearch#34912
dliappis added a commit to elastic/rally that referenced this pull request Oct 29, 2018
The ccr-stats API endpoint has been updated in [1]; update the
telemetry device and tests to align with the changes introduced.

[1] elastic/elasticsearch#34912

Relates #590
kcm pushed a commit that referenced this pull request Oct 30, 2018
* Changed the auto follow stats to also include follow stats.
* Renamed the auto follow stats api to stats api and changed its url path
  from `/_ccr/auto_follow/stats` `/_ccr/stats`.
* Removed `/_ccr/stats` url path for the follow stats api, which makes
  the index parameter a required parameter.
* Fixed docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/CCR Issues around the Cross Cluster State Replication features >non-issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants