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

Revert beat types changes 2x #3560

Merged

Conversation

dreamer-89
Copy link
Member

@dreamer-89 dreamer-89 commented Jun 10, 2022

Description

This PR reverts the temporary changes added to support the typed requests from filebeat. With version spoofing change, these changes are no longer required.
This reverts following PRs from 2.0

  1. f48043b
  2. a93ff13

Issues Resolved

#3484

Related

069948a

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dreamer-89 dreamer-89 requested review from a team and reta as code owners June 10, 2022 18:56
@dreamer-89 dreamer-89 changed the title Revert beat types changes 2x [WIP] Revert beat types changes 2x Jun 10, 2022
kotwanikunal
kotwanikunal previously approved these changes Jun 10, 2022
@dreamer-89
Copy link
Member Author

Verifying changes, please do not merge.

@dreamer-89 dreamer-89 requested a review from nknize June 10, 2022 19:00
@kotwanikunal kotwanikunal dismissed their stale review June 10, 2022 19:02

Waiting for verification

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success bbda825
Log 5897

Reports 5897

@dreamer-89
Copy link
Member Author

Verified changes on 2.0 PR here

@nknize nknize merged commit 55d50dd into opensearch-project:2.x Jun 10, 2022
@dreamer-89 dreamer-89 changed the title [WIP] Revert beat types changes 2x Revert beat types changes 2x Jun 10, 2022
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 this pull request may close these issues.

5 participants