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

Remove Obsolete BWC Serialization Logic #39879

Merged

Conversation

original-brownbear
Copy link
Member

  • We only support 7.0+ now in 8.0 => all BwC serialization logic pertaining to 6.x and older is dead code

* We only support 7.0+ now in 8.0 => all BwC serialization logic pertaining to 6.x and older is dead code
@original-brownbear original-brownbear added >non-issue :Core/Infra/Core Core issues without another label v8.0.0 labels Mar 9, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@original-brownbear
Copy link
Member Author

Jenkins run elasticsearch-ci/1 (checkstyle ran into a SO exception)

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for doing this.

@original-brownbear
Copy link
Member Author

@jasontedor np + thanks for reviewing :)

@original-brownbear original-brownbear merged commit 6621ba5 into elastic:master Mar 9, 2019
@original-brownbear original-brownbear deleted the cleanup-dead-bwc branch March 9, 2019 19:36
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this pull request Mar 10, 2019
* Same as elastic#39879, all Bwc logic related to pre-7.0 versions is obsolete in master (8.0) now
original-brownbear added a commit that referenced this pull request Mar 13, 2019
* Remove Obsolete BwC Serialization Logic

* Same as #39879, all Bwc logic related to pre-7.0 versions is obsolete in master (8.0) now
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this pull request Mar 14, 2019
* Same as elastic#39879 and elastic#39883 <- no need for BwC logic that covers pre-7.0 versions in 8.0
original-brownbear added a commit that referenced this pull request Mar 14, 2019
* Same as #39879 and #39883 <- no need for BwC logic that covers pre-7.0 versions in 8.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >non-issue v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants