Skip to content

Conversation

@GlenRSmith
Copy link
Contributor

(I ought to have done this in my last PR. Mea culpa.)

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS that we support?
  • If you are submitting this code for a class then read our policy for that.
(I ought to have done this in my last PR. Mea culpa.)
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.

@jasontedor jasontedor merged commit e9dfb2a into master Jul 17, 2017
jasontedor pushed a commit that referenced this pull request Jul 17, 2017
When simulating an ingest pipeline against an existing pipeline, the _source field is required to wrap each doc. This commit fixes another example in the docs that is missing this. Relates #25743, relates 8d314dd
jasontedor pushed a commit that referenced this pull request Jul 17, 2017
When simulating an ingest pipeline against an existing pipeline, the _source field is required to wrap each doc. This commit fixes another example in the docs that is missing this. Relates #25743, relates 84e854d
@jasontedor jasontedor deleted the GlenRSmith-patch-3-1 branch July 17, 2017 06:19
@clintongormley clintongormley added the >docs General docs changes label Jul 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes

4 participants