[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Open (was: Patch Available) Colin, looks like you are having some weird line ending problem... patch is not applying. > 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: Zooinspector-patch.patch, zooInspectorChanges.patch, > zooInspectorChanges.patch, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) > 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: Zooinspector-patch.patch, zooInspectorChanges.patch, > zooInspectorChanges.patch, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: Zooinspector-patch.patch Corrected error and resubmitted file > 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: Zooinspector-patch.patch, zooInspectorChanges.patch, > zooInspectorChanges.patch, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: (was: Zooinspector-patch.patch) > 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, zooInspectorChanges.patch, > ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Open (was: Patch Available) patch failed > 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, zooInspectorChanges.patch, > ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) rebuilt and resubmitted patch file > 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: Zooinspector-patch.patch, zooInspectorChanges.patch, > zooInspectorChanges.patch, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: Zooinspector-patch.patch Resubmit > 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: Zooinspector-patch.patch, zooInspectorChanges.patch, > zooInspectorChanges.patch, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Open (was: Patch Available) patch failed > 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, zooInspectorChanges.patch, > ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: zooInspectorChanges.patch @author tags have been removed and patch file line endings have been converted to UNIX format > 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, zooInspectorChanges.patch, > ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) patch resubmitted with fixes > 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, zooInspectorChanges.patch, > ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Open (was: Patch Available) cancelling patch - please address the @author issues (remove them), the javadoc warning seems to be some issue with the link being accessed (not with the change itself). > 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, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Patch Available (was: Open) > 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, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Attachment: ZOOKEEPER-756.patch Looks like line endings are the problem. The current source in svn has some lines with ^M line endings and this seems to be messing up the patch application (at least under unix). I tried it myself and had similar problems (I'm on unix). I then used dos2unix to convert all the zooinspector files to unix line endings, then reapplied Colin's patch and it patched fine. I created a new patch file based on this, which I'm now attaching. Please review this updated patch and make sure I didn't miss anything > 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, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Open (was: Patch Available) > 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, ZOOKEEPER-756.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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) Patch submitted - see previous comment > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: (was: zooInspectorChanges.patch) > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: zooInspectorChanges.patch I have recreated the patch file. I have done this a few times now and I'm not sure what I'm doing wrong so I've explained what I'm doing with TortoiseSVN below, if any of this is wrong could you let me know so that I can recreate it correctly: # Navigated to the repository folder in Windows Explorer # Right-clicked the repository folder and selected TortoiseSVN --> Create Patch... # Selected the files I have changed in the dialog and clicked OK # selected a location and name for the patch file and clicked Save. The repository is set to check out from the following URL: http://svn.apache.org/repos/asf/hadoop/zookeeper/trunk > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Open (was: Patch Available) The patch is failing to apply, please update against the latest source base and resubmit, thanks! > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Giridharan Kesavan updated ZOOKEEPER-756: - Status: Patch Available (was: Open) > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Giridharan Kesavan updated ZOOKEEPER-756: - Status: Open (was: Patch Available) re-submiting to hudson > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Patch Available (was: Open) > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Open (was: Patch Available) > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: zooInspectorChanges.patch recreated the patch file > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: (was: zooInspectorChanges.patch) > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-756: --- Status: Open (was: Patch Available) Cancelling - fails to patch against current codebase. Colin - could you resubmit? Would love to get this in. According to the console output it's failing to patch against the current trunk. > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: zooInspectorChanges.patch New patch file, following failed one > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) Submitted a new patch file > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: (was: zooInspectorChanges.patch) > 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 > > > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Open (was: Patch Available) Patch did not work, cancelling to submit another > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Status: Patch Available (was: Open) Fix Version/s: 3.4.0 I have attached a patch file to this issue which contains the requested changes. You now need to pass in two program arguments: the location of the config folder and the location of the icons folder. > 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.
[jira] Updated: (ZOOKEEPER-756) some cleanup and improvements for zooinspector
[ https://issues.apache.org/jira/browse/ZOOKEEPER-756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Goodheart-Smithe updated ZOOKEEPER-756: - Attachment: zooInspectorChanges.patch pathc file with requested changes > 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 > 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.