-
Notifications
You must be signed in to change notification settings - Fork 993
[DOCS] Add missing items to 8.0.0-rc1 release notes #1863
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
[DOCS] Add missing items to 8.0.0-rc1 release notes #1863
Conversation
- Update log4j version | ||
https://github.com/elastic/elasticsearch-hadoop/pull/1828[#1828] |
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.
@masseyke @jbaiera This isn't in the list at #1859 (comment), but I think it should be included here.
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.
Sounds good to me. We don't actually ship log4j, so it's just a compile-time dependency. But probably worth mentioning even that since the log4j bug was such a high-profile thing.
=== Bug fixes | ||
|
||
Core:: | ||
- Resolve `saveToEs` saves case classes fields with `NULL` values |
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.
These two in the Core
section are actually both spark (I mislabeled them).
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.
Thanks. I moved them under Spark
with 06ac86e.
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
Thanks for spotting this and the review @masseyke! |
When preparing the 8.0.0-rc1 release notes in #1859, I excluded PRs with the
v7.17
label. However, 7.17 hasn't been released yet so those PRs should be included. This adds the missing PRs.