[
https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12894659#action_12894659
]
Hadoop QA commented on ZOOKEEPER-756:
-------------------------------------
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12450513/zooInspectorChanges.patch
against trunk revision 980576.
+1 @author. The patch does not contain any @author tags.
-1 tests included. The patch doesn't appear to include any new or modified
tests.
Please justify why no tests are needed for this patch.
-1 patch. The patch command could not apply the patch.
Console output:
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/98/console
This message is automatically generated.
> some cleanup and improvements for zooinspector
> ----------------------------------------------
>
> Key: ZOOKEEPER-756
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-756
> Project: Zookeeper
> Issue Type: Improvement
> Components: contrib
> Affects Versions: 3.3.0
> Reporter: Thomas Koch
> Assignee: Colin Goodheart-Smithe
> Fix For: 3.4.0
>
> Attachments: zooInspectorChanges.patch
>
>
> Copied from the already closed ZOOKEEPER-678:
> * specify the exact URL, where the icons are from. It's best to include the
> link also in the NOTICE.txt file.
> It seems, that zooinspector finds it's icons only if the icons folder is in
> the current path. But when I install zooinspector as part of the Zookeeper
> Debian package, I want to be able to call it regardless of the current path.
> Could you use getRessources or something so that I can point to the icons
> location from the wrapper shell script?
> Can I place the zooinspector config files in /etc/zookeeper/zooinspector/ ?
> Could I give zooinspector a property to point to the config file location?
> There are several places, where viewers is missspelled as "Veiwers". Please
> do a case insensitive search for "veiw" to correct these. Even the config
> file "defaultNodeVeiwers.cfg" is missspelled like this. This has the
> potential to confuse the hell out of people when debugging something!
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
