python-elasticsearch-logger icon indicating copy to clipboard operation
python-elasticsearch-logger copied to clipboard

Published 20 hours ago • verified publisher icon cmanaha

Remove the declaring the type as it is deprecated

Open werebear73 opened this issue 4 years ago • 11 comments

This is resolve #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 avatar Oct 16 '20 22:10 werebear73

@cmanaha are there any comments or needs to get this pull request accepted?

werebear73 avatar Nov 16 '20 15:11 werebear73

Would be great to have this merged. This change works for me.

javiermatos avatar Nov 16 '20 22:11 javiermatos

+1 would be great to have those issue resolved

simonjk avatar Nov 17 '20 13:11 simonjk

Can we get this merged, please?

martinschi avatar Feb 10 '21 09:02 martinschi

Has anyone heard from the project leader?

werebear73 avatar Sep 21 '21 19:09 werebear73

Hey @werebear73

The original author (Carlos) and I talked a few weeks ago and he gave me the permissions to help bring the code and package up to date for everyone.

Can we setup a meeting this week to review your merge request again? 🤓

Adrian

drumadrian avatar Sep 21 '21 19:09 drumadrian

Sure

werebear73 avatar Sep 21 '21 21:09 werebear73

How about tomorrow at 8am PDT for you and anyone else that wants to chat?

8pm PDT or during the workday is good for me too.

If tomorrow works we can use this Chime meeting info:

You have been invited to an online meeting, powered by Amazon Chime.

Click to join the meeting: https://chime.aws/2764064532
Meeting ID: 2764 06 4532
A headset is recommended or you may use your computer’s microphone and speakers.

Call in using your phone:
United States Toll-Free (1): +1 855-552-4463
Meeting ID: 2764 06 4532
One-click Mobile Dial-in (United States (1)): +1 206-462-5569,,,2764064532#
United States (1): +1 206-462-5569
International: https://chime.aws/dialinnumbers/
Dial-in attendees must enter *7 to mute or unmute themselves.

Download Amazon Chime at https://aws.amazon.com/chime/download
For information about creating an Amazon Chime account, see https://aws.amazon.com/chime/getting-started

drumadrian avatar Sep 21 '21 22:09 drumadrian

The real question is whether it makes sense to try to fix this project when there's ECS-compliant alternative which seems to be maintained: https://github.com/IMInterne/python-elasticsearch-ecs-logger. I moved all my projects to it and it was mostly plug-and-play replacement and besides following ECS schema it also fixed quite a few bugs on the way, including some log messages getting lost on their way to Elasticsearch.

martinschi avatar Sep 22 '21 06:09 martinschi

Reopened after decision with Adrian

werebear73 avatar Sep 22 '21 15:09 werebear73

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)

elijahxb avatar May 04 '22 16:05 elijahxb