cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "mck (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CASSANDRA-11706) Tracing payload not passed through newSession(..)
Date Mon, 29 Aug 2016 12:32:20 GMT

     [ https://issues.apache.org/jira/browse/CASSANDRA-11706?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

mck updated CASSANDRA-11706:
----------------------------
    Resolution: Fixed
        Status: Resolved  (was: Ready to Commit)

Committed in https://git1-us-west.apache.org/repos/asf?p=cassandra.git;a=commit;h=f0c94a43f23d338cbbb3a4420e9f296484a10dc1

> Tracing payload not passed through newSession(..)
> -------------------------------------------------
>
>                 Key: CASSANDRA-11706
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-11706
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: mck
>            Assignee: mck
>            Priority: Minor
>              Labels: tracing
>             Fix For: 3.x
>
>         Attachments: trunk-11706.txt
>
>
> Caused by CASSANDRA-10392
> There's a small bug in https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/tracing/Tracing.java#L153
> {noformat}
>     public UUID newSession(UUID sessionId, Map<String,ByteBuffer> customPayload)
>     {
>         return newSession(sessionId, TraceType.QUERY, Collections.EMPTY_MAP);
>     }{noformat}
> in that it passes on an {{EMPTY_MAP}} instead of the {{customPayload}}.
> I've marked this as "minor" as custom tracing plugins can easily enough workaround it
by also overriding the {{newSession(UUID sessionId, Map<String,ByteBuffer> customPayload)}}
method.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message