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

Specifying types in bulk requests is deprecated #76

Open
luke-han opened this issue Jul 8, 2020 · 5 comments · May be fixed by #79
Open

Specifying types in bulk requests is deprecated #76

luke-han opened this issue Jul 8, 2020 · 5 comments · May be fixed by #79

Comments

@luke-han
Copy link

luke-han commented Jul 8, 2020

I am using CMRESHandler with ES 7.5.1.

From ES 7, type is deprecated

When CMRESHandler sends logs with bulk operation, [types removal] Specifying types in bulk requests is deprecated. warning is printed.

Forcing to set es_doc_type is cause of this issue.

This line may need to be deleted.

werebear73 added a commit to werebear73/python-elasticsearch-logger that referenced this issue Oct 16, 2020
This is resolve cmanaha#76 (Specifying types in bulk requests is deprecated) which results in the following warning.

ElasticsearchDeprecationWarning: [types removal] Specifying types in bulk requests is deprecated.
@werebear73 werebear73 linked a pull request Oct 16, 2020 that will close this issue
@mkhadher
Copy link

mkhadher commented Dec 8, 2021

This's fixed in fork https://github.com/mkhadher/python-elasticsearch-logger

@elijahxb
Copy link

elijahxb commented May 4, 2022

Maybe, we could solve this problem by passing a None value to type field es_doc_type
just like this:
CMRESHandler(xxx=xxx, es_doc_type=None)

@drumadrian
Copy link
Collaborator

Hi @everyone !

Hey @mkhadher

I would like to meet and talk about the python-elasticsearch-logger that we all use. I think we can merge all of the best features and update PyPi. I work at AWS and talked with Carlos about this.

Python maintainer,

Adrian Drummond
[email protected]
[email protected]

@constructor-igor
Copy link

Hi.
please, could you provide a solution for the issue?

@drumadrian
Copy link
Collaborator

I should have done this by now but I switched jobs and I am going to prioritize this in 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants