[jira] [Comment Edited] (CASSANDRA-13307) The specification of protocol version in cqlsh means the python driver doesn't automatically downgrade protocol version.

["mck (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CASSANDRA-13307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15962001#comment-15962001 ] 

mck edited comment on CASSANDRA-13307 at 4/9/17 12:17 PM:
----------------------------------------------------------

+1
The code looks good and makes sense to me.

Test results are 
|| Branch || Unit Tests || DTests ||
| [trunk|https://github.com/apache/cassandra/pull/96/commits/c36a4e5547af3967976144f7b553d70873503f77] | [asf jenkins|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-testall/3] [circleci|https://circleci.com/gh/michaelsembwever/cassandra/2] | [dtest|] |

^i don't have access to put builds/branches on cassci, and i understand from [this|http://cassci.datastax.com/userContent/cassci-usage.html], and the mailing list, no one will be added in lei of waiting for the new ASF build infra.

For the meantime, the unit tests all pass on my machine.


was (Author: michaelsembwever):
+1
The code looks good and makes sense to me.

Test results are 
|| Branch || Unit Tests || DTests ||
| [trunk|https://github.com/apache/cassandra/pull/96/commits/c36a4e5547af3967976144f7b553d70873503f77] | [testall|] | [dtest|] |

^i don't have access to put builds/branches on cassci, and i understand from [this|http://cassci.datastax.com/userContent/cassci-usage.html], and the mailing list, no one will be added in lei of waiting for the new ASF build infra.

For the meantime, the unit tests all pass on my machine.

> The specification of protocol version in cqlsh means the python driver doesn't automatically downgrade protocol version.
> ------------------------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-13307
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-13307
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Tools
>            Reporter: Matt Byrd
>            Assignee: Matt Byrd
>            Priority: Minor
>             Fix For: 3.11.x
>
>
> Hi,
> Looks like we've regressed on the issue described in:
> https://issues.apache.org/jira/browse/CASSANDRA-9467
> In that we're no longer able to connect from newer cqlsh versions
> (e.g trunk) to older versions of Cassandra with a lower version of the protocol (e.g 2.1 with protocol version 3)
> The problem seems to be that we're relying on the ability for the client to automatically downgrade protocol version implemented in Cassandra here:
> https://issues.apache.org/jira/browse/CASSANDRA-12838
> and utilised in the python client here:
> https://datastax-oss.atlassian.net/browse/PYTHON-240
> The problem however comes when we implemented:
> https://datastax-oss.atlassian.net/browse/PYTHON-537
> "Don't downgrade protocol version if explicitly set" 
> (included when we bumped from 3.5.0 to 3.7.0 of the python driver as part of fixing: https://issues.apache.org/jira/browse/CASSANDRA-11534)
> Since we do explicitly specify the protocol version in the bin/cqlsh.py.
> I've got a patch which just adds an option to explicitly specify the protocol version (for those who want to do that) and then otherwise defaults to not setting the protocol version, i.e using the protocol version from the client which we ship, which should by default be the same protocol as the server.
> Then it should downgrade gracefully as was intended. 
> Let me know if that seems reasonable.
> Thanks,
> Matt



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)