-
Notifications
You must be signed in to change notification settings - Fork 875
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
JAVA-3083 Doc updates for new features in 4.17.0 #1677
Conversation
@hhughes I believe I covered everything we aimed to include with this ticket. If you see something is missing please note it here and I'll fix it. |
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
Co-authored-by: Jamie Gillenwater <jamie.gillenwater@datastax.com>
Co-authored-by: Jamie Gillenwater <jamie.gillenwater@datastax.com>
Co-authored-by: Jamie Gillenwater <jamie.gillenwater@datastax.com>
Co-authored-by: Jamie Gillenwater <jamie.gillenwater@datastax.com>
upgrade_guide/README.md
Outdated
```java | ||
try (CqlSession session = new CqlSessionBuilder().withLocalDatacenter("datacenter1").build()) { | ||
|
||
session.execute("drop keyspace if exists test"); |
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.
nit: is the drop keyspace line neccessary for this example?
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.
Perhaps not strictly necessary but I'd argue it's certainly a good practice. Including it here communicates the idea that this example should reset its state before it runs; I want to cut off any cases of the test code not working because the user already had a "test" keyspace defined.
Co-authored-by: Jamie Gillenwater <jamie.gillenwater@datastax.com>
Co-authored-by: Jamie Gillenwater <jamie.gillenwater@datastax.com>
No description provided.