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

Sean Busbey commented on ACCUMULO-1870:
---------------------------------------

+1 (non-binding) looks good.

> Functional tests don't configure Monitor log 
> ---------------------------------------------
>
>                 Key: ACCUMULO-1870
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1870
>             Project: Accumulo
>          Issue Type: Bug
>          Components: test
>    Affects Versions: 1.4.5, 1.5.1
>            Reporter: Sean Busbey
>            Assignee: Ryan Fishel
>            Priority: Trivial
>              Labels: newbie, patch
>             Fix For: 1.4.4, 1.5.0
>
>         Attachments: ACCUMULO-1870.patch
>
>
> In reviewing ACCUMULO-1658, we missed one line that has an extra level of 
> "conf":
> test/system/auto/TestUtils.py line 50
> {code}
> LOG_MONITOR = os.path.join(ACCUMULO_CONF_DIR, 'conf', 'monitor_logger.xml')
> {code}
> I believe the net result is tests end up using the original monitor logger 
> file.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to