-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[WIP] Allow client restart if required #19554
base: main
Are you sure you want to change the base?
Conversation
@mohityadav766 lets not add this. We shouldn't be introducing this type of change. it won't help us catch actual bug and fix it. |
Coming up with the actual change , need to introduce ClientHttpClirntAshnc with some Safe consumer class(as per the es docs) , upgrading es clients will fix this , but that would be bigger change |
lets not worry about this change. Lets only introduce the metric to monitor it |
Quality Gate passed for 'open-metadata-ingestion'Issues Measures |
Describe your changes:
I worked on ... because ...
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>