Ian Maxon has posted comments on this change.

Change subject: Various fixes for Windows NCService
......................................................................


Patch Set 4:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1518/4/asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.bat
File asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.bat:

PS4, Line 55: 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
            : 
> Why this deletion?  Did you test appassembler with spaces in JAVA_HOME?
Because JAVA_HOME isn't set by default from what I've seen. I'm just letting it 
use whatever "java" points to. The default path has a space for sure 
(C:\Program Files\...)


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1518
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If7af80292bddd5191d83bac18f0a518621d7db71
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <ima...@apache.org>
Gerrit-Reviewer: Ian Maxon <ima...@apache.org>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to