[
https://issues.apache.org/jira/browse/ACCUMULO-4162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196076#comment-15196076
]
Christopher Tubbs commented on ACCUMULO-4162:
---------------------------------------------
Replacing
{code}
ZOOKEEPER_VERSION=${ZOOKEEPER_VERSION##$ZOOKEEPER_HOME/zookeeper-}
{code}
with
{code}
ZOOKEEPER_VERSION=${ZOOKEEPER_VERSION##*/zookeeper-}
{code}
is probably good.
I'm also not a fan of the needing to escape the comparison operator. We should
use double brackets. It's probably a good idea to just run the whole script
through shellcheck, too.
> Invalid ZooKeeper version warning message
> -----------------------------------------
>
> Key: ACCUMULO-4162
> URL: https://issues.apache.org/jira/browse/ACCUMULO-4162
> Project: Accumulo
> Issue Type: Bug
> Components: scripts
> Affects Versions: 1.7.1
> Reporter: Josh Elser
>
> Saw this when running Sven's docker image with ZooKeeper-3.4.8:
> {noformat}
> WARN : Using Zookeeper /zookeeper-3.4.8/zookeeper-3.4.8. Use version 3.4.0
> or greater. Older versions may not work reliably.
> {noformat}
> 3.4.8 > 3.4.0. Looks like the version detection is a bit busted.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)