[
https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098860#comment-13098860
]
Hadoop QA commented on ZOOKEEPER-756:
-------------------------------------
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12493295/ZOOKEEPER-756.patch
against trunk revision 1165443.
-1 @author. The patch appears to contain 5 @author tags which the
Zookeeper community has agreed to not allow in code contributions.
-1 tests included. The patch doesn't appear to include any new or modified
tests.
Please justify why no new tests are needed for this
patch.
Also please list what manual steps were performed to
verify this patch.
+1 javadoc. The javadoc tool did not generate any warning messages.
-1 javac. The patch appears to cause tar ant target to fail.
+1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9)
warnings.
+1 release audit. The applied patch does not increase the total number of
release audit warnings.
+1 core tests. The patch passed core unit tests.
+1 contrib tests. The patch passed contrib unit tests.
Test results:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/507//testReport/
Findbugs warnings:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/507//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/507//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: Thomas Koch
> Labels: zooinspector
> Fix For: 3.5.0
>
> Attachments: ZOOKEEPER-756.patch, ZOOKEEPER-756.patch,
> Zooinspector-patch.patch, zooInspectorChanges.patch, 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.
For more information on JIRA, see: http://www.atlassian.com/software/jira
