accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From joshelser <...@git.apache.org>
Subject [GitHub] accumulo issue #255: ACCUMULO-4365: Fix trace test in ShellServerIT
Date Fri, 28 Apr 2017 15:18:37 GMT
Github user joshelser commented on the issue:

    https://github.com/apache/accumulo/pull/255
  
    Aha!
    
    > The following spans are not rooted (probably due to a parent span of length 0ms):
    
    I don't think this test is even validating the correct thing. That span is supposed to
be under..
    
    ```
     1010+168      shell@canx2um-78286w.infosec.tycho.ncsc.mil close
        2+169        shell@canx2um-78286w.infosec.tycho.ncsc.mil BinMutations 1
        2+169          shell@canx2um-78286w.infosec.tycho.ncsc.mil binMutations
    ```
    
    The test is really just verifying that we non-rooted spans are still logged (which is
definitely not the intent). It seems more like ACCUMULO-1755 broke the span hierarchy in the
TSBW.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message