Skip to content

Added support for "update" action - #10

Open
sxmichael wants to merge 3 commits into
logstash-plugins:mainfrom
sxmichael:master
Open

Added support for "update" action#10
sxmichael wants to merge 3 commits into
logstash-plugins:mainfrom
sxmichael:master

Conversation

@sxmichael

@sxmichael sxmichael commented Aug 6, 2016

Copy link
Copy Markdown

Includes doc and doc_as_upsert support, scripted updated and upsert. The funcionality is activated by passing parse_update=true to codec.

@karmi

karmi commented Aug 6, 2016

Copy link
Copy Markdown

Hi @sxmichael, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

@sxmichael

sxmichael commented Aug 6, 2016

Copy link
Copy Markdown
Author

This pull request actually replaces previous pull request #6 by @kiryam

@sxmichael
sxmichael force-pushed the master branch 2 times, most recently from 5acb32d to 957a19d Compare August 7, 2016 10:59
@sxmichael
sxmichael force-pushed the master branch 2 times, most recently from fa1b786 to 09a2cae Compare August 7, 2016 20:28
Including doc and doc_as_upsert support, scripted updated and upsert.
@sxmichael

Copy link
Copy Markdown
Author

@karmi can you please check if everything is good now?

@suyograo

Copy link
Copy Markdown
Contributor

@sxmichael CLA is good. We'll review this patch soon. Thanks for contributing to Logstash

@sxmichael

Copy link
Copy Markdown
Author

@karmi @suyograo any update?...

@sxmichael

Copy link
Copy Markdown
Author

@karmi @suyograo any update?...

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