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

Bret McGuire commented on CASSPYTHON-29:
----------------------------------------

[~swsekar] Can you elaborate a bit on why this change would be necessary?  
APPLICATION_NAME and APPLICATION_VERSION were used as part of the Insights 
offering from DataStax.  That effort has long been defunct... we just haven't 
been as vigorous about removing support for it as we'd like.  It's worth noting 
that the [native protocol 
spec|https://github.com/apache/cassandra/blob/trunk/doc/native_protocol_v5.spec#L462-L491]
 doesn't identify very many specific keys for what can appear in STARTUP 
messages (and it definitely doesn't include either APPLICATION_NAME or 
APPLICATION_VERSION).  I'll also note that we're already heading in the other 
direction; we're starting to work on version 5.0.0 of the Java driver and hope 
to include [this PR|https://github.com/apache/cassandra-java-driver/pull/2038] 
which removes Insights support entirely (including the removal of the 
withApplicationName() and withApplicationVersion() methods referenced in your 
original comment).  The C/C++ driver is a bit behind on this front but I would 
anticipate something similar for a 3.0.0 release of that driver as well.

> Send `APPLICATION_NAME` and `APPLICATION_VERSION` in the STARTUP message
> ------------------------------------------------------------------------
>
>                 Key: CASSPYTHON-29
>                 URL: https://issues.apache.org/jira/browse/CASSPYTHON-29
>             Project: Apache Cassandra Python driver
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Swathi Sekar
>            Assignee: Swathi Sekar
>            Priority: Normal
>
> *Summary*
> The driver should send the client's application name and version as 
> `APPLICATION_NAME` /
> `APPLICATION_VERSION` options in the STARTUP message.
> This brings the Python driver in line with the C++ driver (since 2.11.0, via
> `cass_cluster_set_application_name()` / 
> `cass_cluster_set_application_version()`) and the Java
> driver (since 4.4.0, via `SessionBuilder.withApplicationName` / 
> `withApplicationVersion`).
> *More info*
> `Cluster` already accepts `application_name`, `application_version` but they 
> are
> documented as Insights-only and are consumed solely by
> `cassandra/datastax/insights/reporter.py`. They are never sent over the 
> native protocol.
> This change would add them to the STARTUP options map.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to