[
https://issues.apache.org/jira/browse/CASSANDRA-16613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17334907#comment-17334907
]
Ekaterina Dimitrova edited comment on CASSANDRA-16613 at 4/28/21, 5:54 PM:
---------------------------------------------------------------------------
Thank you [~samt] for the quick response.
I will accommodate all your suggestions.
I was wondering whether we always want to loop all supported versions or in
some cases we need only latest one or as in the case with the _PagingStateTest_
- a few older ones and special mixed cases, etc. So you just gave me that
response.
Thanks one more time for looking into it, I will submit a patch for review
later today.
was (Author: e.dimitrova):
Thank you [~samt] for the quick response.
I will accommodate all your suggestions.
I was wondering whether we always want to loop all supported versions or in
some cases we need only latest one or as in the case with the PagingStateTest-
a few older ones and special mixed cases, etc. So you just gave me that
response.
Thanks one more time for looking into it, I will submit a patch for review
later today.
> ProtocolVersion.V4 is still used in places in the code
> ------------------------------------------------------
>
> Key: CASSANDRA-16613
> URL: https://issues.apache.org/jira/browse/CASSANDRA-16613
> Project: Cassandra
> Issue Type: Bug
> Reporter: Ekaterina Dimitrova
> Assignee: Ekaterina Dimitrova
> Priority: Normal
> Fix For: 4.0-rc
>
>
> While working on CASSANDRA-16567, [~adelapena] observed that
> _ProtocolVersion.V4_ is used in _ViewTest_.
> I decided to do a quick grep and observed a list of places where we still
> refer to V4 and it seems at least in many of the tests that was left not
> intentionally.
> This ticket is to verify the usage of _ProtocolVersion.V4_ in the codebase
> and bump it to V5 or default version, similar to what was done in
> CASSANDRA-16567, wherever there is a need.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
