Fix bug with doc field in update - #6
Conversation
|
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'. |
|
@suyograo can you review it, please? |
|
@kiryam I'm not sure why there are merge conflicts here. Can you please rebase and see what happens? We can't merge it with the branch conflicts. |
|
@untergeek done. |
|
@untergeek I am sorry, but what about merge? |
added support for upsert/script in update
|
@karmi I accepted pull request kiryam#1 |
|
@untergeek @karmi hi guys, any update on this? |
|
bump |
|
There are some conflicts in the branch that need to be resolved before we can merge. |
|
@kiryam could you please rebase/merge changes from master? |
|
i've created new pull request #10 with exactly same changes, in addition of config parameters which control wether to use this functionallity or not. |
|
Is this fixed, if not any work arounds? |
https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-bulk.html
"Update" should generate event without doc field.