[CASSANDRA-15460] Fix missing call to enable RPC after native transport is started in in-jvm dtests

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 2.2.16, 3.0.20, 3.11.6, 4.0, 4.0-alpha3
• Component/s: Test/dtest/java
• Labels: pull-request-available
• Bug Category: Correctness - Test Failure
• Severity: Normal
• Complexity: Low Hanging Fruit
• Discovered By: Unit Test
• Platform: All
• Since Version: 2.2.15

Source Control Link

https://github.com/apache/cassandra/commit/f7ee96c74f783b42e520d26d278eafaca2a59678

Test and Documentation Plan

Patch in Github PR #399 includes new unit test for counter updates.

• Assignee: Doug Rohrer
• Reporter: Doug Rohrer

Authors

Doug Rohrer

Reviewers

Alex Petrov, David Capwell, Yifan Cai

• Created: 19/Dec/19 12:08
• Updated: 27/Aug/20 15:08
• Resolved: 29/Jan/20 18:27

Description

When starting the native transport, the original patch missed the step of calling StorageService.instance.setRpcReady(true);. This appears to only be required for counter columns, but without it you can't update a counter value.

We should add this call after starting up the native transport, and set it to false during the shutdown sequence to mimic the production code.

Issue Links

• links to: GitHub Pull Request #399