-
Notifications
You must be signed in to change notification settings - Fork 25
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
Update DSE_6.9_Release_Notes.md #172
Conversation
Append new information from a ticket (DSP-24431) that was merged before release but it was not closed.
DSE_6.9_Release_Notes.md
Outdated
@@ -19,6 +19,7 @@ If you're developing applications, please refer to the [Java Driver documentatio | |||
|
|||
## 6.9.4 DSE Core | |||
* Updated the JDK used to build DSE to versions `8u422` and `11.0.24`. (DSP-23997) | |||
* Changed the MAX_HEAP_SIZE to avoid OOM errors with new Java11 Garbage Collector. The change is done in the following scrips: bin/dse, bin/dsetool, dse-db/bin/nodetool , resources/cassandra/bin/nodetool . (DSP-24431) |
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.
What about?
* Changed the default MAX_HEAP_SIZE in `dse`, `dsetool` `nodetool`, and `nodetool` scripts to avoid OOM errors with new Java 11 Garbage Collector. (DSP-24431)
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.
I think that will make it harder to understand if it's a typo in > nodetool
, and nodetool
scripts < or we are referring a two different scripts
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.
good point! does our packages provide two nodetool
scripts? If so, then we should use the paths from the package not from the source code.
remove dse-db/nodetool as it doesn't show up in the package we provide
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.
Left some more minor suggestions. No need for further review rounds.
DSE_6.9_Release_Notes.md
Outdated
@@ -19,6 +19,7 @@ If you're developing applications, please refer to the [Java Driver documentatio | |||
|
|||
## 6.9.4 DSE Core | |||
* Updated the JDK used to build DSE to versions `8u422` and `11.0.24`. (DSP-23997) | |||
* Changed the MAX_HEAP_SIZE to avoid OOM errors with new Java11 Garbage Collector. The change is done in the following scrips: bin/dse, bin/dsetool, resources/cassandra/bin/nodetool . (DSP-24431) |
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.
We could use markdown formatting for the scripts and have a space between "Java" and "11".
Append new information from a ticket (DSP-24431) that was merged before release but it was not closed.
Release Notes Automation
If you name your pull-request as "Product x.y.z Release ...", after merging the
PR, a GitHub Action will automatically create a product version tag "product-x.y.z".
Supported product names are:
Version supports 3 sets or 4 sets of digits.