-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Release notes for 7.2 #10863
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
Release notes for 7.2 #10863
Conversation
@danhermann Can you check the release notes around the changes made for GA of the native support for Java plugins to make sure I have represented everything correctly? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Java plugins note looks good aside from one more breaking change: The return type on the Context::getDlqWriter
method was changed to an interface to decouple it from any specific DLQ implementation.
@danhermann Is that this change, or is there a different one? |
Yes, that's the one. I see now that it's mentioned further down, but it was requested that we specifically call out breaking changes to the Java plugin API since the beta release. |
@danhermann I've update the called out list of changes to native support to include that breaking change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comments in line. Please let me know if you have questions.
@karenzone I think I've fixed the issues that you brought up in the review |
Refer to breaking change in how the DLQ interfaces with native Java plugin support
b13ebee
to
9439287
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Release notes for 7.2 (#10863) * add release notes for 7.3.0 * Fix link formatting
* Release notes for 7.2 (elastic#10863) * add release notes for 7.3.0 * Fix link formatting
* Release notes for 7.2 (elastic#10863) * add release notes for 7.3.0 * Fix link formatting
* Release notes for 7.2 (#10863) * 7.2.1 Release notes (#10959) * Add coming tag to 7.2.1 release notes (#10968) * Updated release notes for 7.2.1 (#10996) * Release notes 7.3.0 (#10993) * Release notes for 7.2 (#10863) * add release notes for 7.3.0 * Fix link formatting * Release notes draft for 7.3.1 (#11038) * Release notes draft for 7.3.2 (#11128) * Update release notes for 7.3.2 * Update releasenotes.asciidoc
* Release notes for 7.2 (#10863) * 7.2.1 Release notes (#10959) * Add coming tag to 7.2.1 release notes (#10968) * Updated release notes for 7.2.1 (#10996) * Release notes 7.3.0 (#10993) * Release notes for 7.2 (#10863) * add release notes for 7.3.0 * Fix link formatting * Release notes draft for 7.3.1 (#11038) * Release notes draft for 7.3.2 (#11128) * Update release notes for 7.3.2 * Update releasenotes.asciidoc
No description provided.