You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2020/01/02 09:36:20 UTC

[GitHub] [zookeeper] maoling commented on a change in pull request #1195: fix zkCli.cmd error

maoling commented on a change in pull request #1195: fix zkCli.cmd error
URL: https://github.com/apache/zookeeper/pull/1195#discussion_r362412744
 
 

 ##########
 File path: bin/zkCli.cmd
 ##########
 @@ -17,7 +17,7 @@ REM limitations under the License.
 setlocal
 call "%~dp0zkEnv.cmd"
 
-ZOO_LOG_FILE=zookeeper-%USERNAME%-cli-%COMPUTERNAME%.log
+set ZOO_LOG_FILE=zookeeper-%USERNAME%-cli-%COMPUTERNAME%.log
 
 set ZOOMAIN=org.apache.zookeeper.ZooKeeperMain
 
 Review comment:
   @wordfeng
   - Yes, this bug has been fixed by [ZOOKEEPER-3453](https://issues.apache.org/jira/browse/ZOOKEEPER-3453)
   - Please change this github description to "`ZOOKEEPER-3453: missing 'SET' in zkCli on windows`"

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services