-
Notifications
You must be signed in to change notification settings - Fork 963
Release note for 4.14.3 #2864
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 note for 4.14.3 #2864
Conversation
| title: Apache BookKeeper 4.14.3 Release Notes | ||
| --- | ||
|
|
||
| Release 4.14.3 fixes an issue with Prometheus metrics that was |
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.
which issue is that? It is not obvious the list below. #2718 was included in 4.14.1, was there something else?
If it deserves being highlighted in the release notes I think it should have a little bit more info describing the problem.
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.
also the release includes many more fixes, so maybe word it as "Release X includes multiple stability, performance, and security fixes along with the fix for Prometheus metrics regression. The regression introduced in release Y resulted in ."
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.
Ah. I am not noticed that. It's generated from scripts and I only changed the Highlights. I will fix that. Thanks
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.
Fixed. PTAL
|
@eolivelli @dlg99 Please take a look at this. The vote has been closed. I need to merge this PR. #2863 need to merge before merging this. Thank you! |
dlg99
left a comment
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 with one spelling correction
Co-authored-by: Andrey Yegorov <8622884+dlg99@users.noreply.github.com>
### Motivation Release note for 4.14.3
Motivation
Release note for 4.14.3
PLEASE MERGE #2863 BEFORE MERGING THIS PR