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

Update DSE_6.9_Release_Notes.md #172

Merged
merged 3 commits into from
Nov 20, 2024
Merged

Update DSE_6.9_Release_Notes.md #172

merged 3 commits into from
Nov 20, 2024

Conversation

roxananeo
Copy link
Contributor

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:

  • DSE
  • OpsCenter
  • Studio
  • Luna Streaming

Version supports 3 sets or 4 sets of digits.

Append new information from a ticket (DSP-24431) that was merged before release but it was not closed.
@@ -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)
Copy link
Member

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)

Copy link
Contributor Author

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

Copy link
Member

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
Copy link
Member

@tiagomlalves tiagomlalves left a 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.

@@ -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)
Copy link
Member

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".

@roxananeo roxananeo merged commit 26153d8 into master Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants