accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-4191) Tracing on client can sometimes lose "sendMutations" events.
Date Fri, 15 Apr 2016 16:54:25 GMT


ASF GitHub Bot commented on ACCUMULO-4191:

Github user joshelser commented on the pull request:
    > For me, ShellServerIT.trace() always fails. I've been banging my head against this
for the past 3 days, trying to get tests to succeed while working on ACCUMULO-4187. I'll take
your word that it doesn't fail for everyone all the time. With this change, ShellServerIT
has succeeded for me the 5 or 6 times I've run it.
    Sorry about this. @billierinaldi had pointed out to me that she thinks ACCUMULO-1755 had
broken this. I didn't know that this one was broken. If our tests are not reliably passing,
that's wrong/bad. Give us a shout before bashing your head next time :)
    > Race conditions are notably difficult to diagnose.
    Yep, that's why I asked :).
    >  I can think of a few ways one might better expose this particular bug under lab
conditions, but any such test would by necessity be timing sensitive, and I do so hate knowingly
writing timing-sensitive tests.
    Yep, totally not asking you to do so. I was looking for some understanding that you 1.
put thought into whether or not this is directly test-able and 2. you had a plausible idea
of why this was sometimes happening. Sounds fine to me.

> Tracing on client can sometimes lose "sendMutations" events.
> ------------------------------------------------------------
>                 Key: ACCUMULO-4191
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Shawn Walker
>            Assignee: Shawn Walker
>            Priority: Minor
> In {{TabletServerBatchWriter.MutationWriter.queueMutations(...)}} (line 736), HTrace
{{Span}}'s are not properly propagated across thread boundaries.  Consequently, tracing doesn't
propagate into {{TabletServerBatchWriter.SendTask.send(...)}}, and so the {{sendMutation}}
event can fail to be logged.
> This can cause {{ShellServerIT.trace(...)}} to fail.

This message was sent by Atlassian JIRA

View raw message