cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Stupp (JIRA)" <>
Subject [jira] [Updated] (CASSANDRA-9429) Revert CASSANDRA-7807 (tracing completion client notifications)
Date Tue, 19 May 2015 20:28:00 GMT


Robert Stupp updated CASSANDRA-9429:
    Attachment: 9429.txt

Attached patch that reverts CASSANDRA-7807 (but leaves the "good side effects" like debug-client,

> Revert CASSANDRA-7807 (tracing completion client notifications)
> ---------------------------------------------------------------
>                 Key: CASSANDRA-9429
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Adam Holmberg
>            Assignee: Robert Stupp
>            Priority: Minor
>             Fix For: 2.2.0 rc1
>         Attachments: 9429.txt
> Follow-on to CASSANDRA-7807, where I opened a discussion without realizing the version
was released:
> I know I'm late to this ticket, but as I began to integrate this feature for the Python
driver, I developed some reservations about its usefulness. I understand the original intent
to avoid polling and eventually simplify trace handling. However, it introduces some complexity
supporting multiple protocol versions, and I'm not sure it will even avoid the need for polling,
due to replication lag.
> Here are some of the things required to take advantage:
> - Extra registration overhead per connection, or dynamically register on first trace
> - Extra book-keeping – trace ID per connection
> - New modes of failure (polling is resilient as the cluster)
> - Event may arrive before trace data is replicated
> - Drivers must still implement polling following event, and for lesser protocol versions
> Given the above, I'm not sure it's worth it to implement to save a few polling round-trips,
especially when it's only client-initiated traces (which will typically not be in a hot path).
> I'm open to discussion on the matter.

This message was sent by Atlassian JIRA

View raw message