Details
-
Bug
-
Status: Resolved
-
Normal
-
Resolution: Fixed
-
None
-
Correctness - Consistency
-
Normal
-
Normal
-
Code Inspection
-
All
-
None
-
Description
I've had some dtests with disabled hints failing.
After investigation it seems that CASSANDRA-17164 moved hint submission on timeout from RequestCallbacks.onExpired to AbstractWriteResponseHandler.onFailure, but it no longer checks if CallbackInfo.shouldHint which checks for StorageProxy.shouldHint which ultimately checks if hinted_handoff_enabled=true.
This could cause some tests which expect hints to be disabled to fail intermittently.