Skip to content

Conversation

martijnvg
Copy link
Member

Added test that verifies that all fields in ShardFollowNodeTaskStatus are mapped in monitoring-es.json.

Added test that verifies that all fields in ShardFollowNodeTaskStatus are
mapped in monitoring-es.json
@martijnvg martijnvg added review :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features labels Sep 13, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

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. This is wonderful, I am so happy that we have this.

@martijnvg martijnvg merged commit b9d0c8f into elastic:master Sep 13, 2018
martijnvg added a commit that referenced this pull request Sep 13, 2018
Added test that verifies that all fields in ShardFollowNodeTaskStatus are
mapped in monitoring-es.json
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Sep 13, 2018
…polling

* elastic/master:
  SQL: Return correct catalog separator in JDBC (elastic#33670)
  [CCR] Add validation for max_retry_delay (elastic#33648)
  [CCR] Add monitoring mapping verification test (elastic#33662)
  CORE: Disable Setting Type Validation (elastic#33660) (elastic#33669)
  Revert "Use serializable exception in GCP listeners (elastic#33657)"
  Adding index refresh (elastic#33647)
  [DOCS] Moves securing-communications to docs (elastic#33640)
  [HLRC][ML] Add ML delete datafeed API to HLRC (elastic#33667)
  Mute testRecoveryWithConcurrentIndexing
  TEST: decrease logging level in the flush test
  DOC: Add SQL section on client applications
  Fix race in global checkpoint listeners test
  Use serializable exception in GCP listeners (elastic#33657)
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants