Skip to content
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

Remove types from watcher docs #38002

Merged
merged 4 commits into from
Jan 30, 2019

Conversation

jakelandis
Copy link
Contributor

Types have been deprecated and this commit removes the documentation
for specifying types in the index action, and search input/transform.

Relates #37594 #35190

Types have been deprecated and this commit removes the documentation
for specifying types in the index action, and search input/transform.

Relates elastic#37594 elastic#35190
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@jtibshirani
Copy link
Contributor

Thanks @jakelandis for these updates! I saw a couple remaining references to types in ack-watch and transform/search.

Copy link
Contributor

@spinscale spinscale left a comment

Choose a reason for hiding this comment

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

There is one more use of doc_type in get-watch.asciidoc on top of what Julie said.. I didnt look in the rest-api/ directory at first

@jakelandis
Copy link
Contributor Author

@jtibshirani @spinscale thanks for the reviews. I totally missed the rest-api/watcher docs and have now included them in 5954cda. The reference in transform/search is for search_type as in query_then_fetch, so no changes there.

Ready for re-review.

Copy link
Contributor

@jtibshirani jtibshirani left a comment

Choose a reason for hiding this comment

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

For transform/search, it looks like document types are mentioned as well (on the lines mentioning request.types and "types" : [ "event" ]). Looks good to me to merge after that's updated!

@jakelandis
Copy link
Contributor Author

@jakelandis
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/2

@jtibshirani
Copy link
Contributor

Sorry @jakelandis, I was reviewing too speedily! Looks good to me.

@jakelandis jakelandis merged commit 6a78b6a into elastic:master Jan 30, 2019
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jan 30, 2019
* master:
  Remove types from watcher docs (elastic#38002)
  Add test coverage for Painless general casting of boolean and Boolean (elastic#37780)
  Fixed test bug, lastFollowTime is null if there are no follower indices.
  Add ECS schema for user-agent ingest processor (elastic#37727) (elastic#37984)
  Extract TransportRequestDeduplication from ShardStateAction (elastic#37870)
  Expose retention leases in shard stats (elastic#37991)
jasontedor added a commit to dnhatn/elasticsearch that referenced this pull request Jan 31, 2019
* elastic/master:
  ILM setPriority corrections for a 0 value (elastic#38001)
  Temporarily disable BWC for retention lease stats (elastic#38049)
  Skip Shrink when numberOfShards not changed (elastic#37953)
  Add dispatching to `HandledTransportAction` (elastic#38050)
  Update httpclient for JDK 11 TLS engine (elastic#37994)
  Reduce flaxiness of ccr recovery timeouts test (elastic#38035)
  Fix ILM status to allow unknown fields (elastic#38043)
  Fix ILM Lifecycle Policy to allow unknown fields (elastic#38041)
  Update verify repository to allow unknown fields (elastic#37619)
  [ML] Datafeed deprecation checks (elastic#38026)
  Deprecate minimum_master_nodes (elastic#37868)
  Remove types from watcher docs (elastic#38002)
  Add test coverage for Painless general casting of boolean and Boolean (elastic#37780)
  Fixed test bug, lastFollowTime is null if there are no follower indices.
  Add ECS schema for user-agent ingest processor (elastic#37727) (elastic#37984)
  Extract TransportRequestDeduplication from ShardStateAction (elastic#37870)
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