Yingyi Bu has posted comments on this change.

Change subject: Fix local cluster stop and erase scripts.
......................................................................


Patch Set 4:

(6 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1549/3//COMMIT_MSG
Commit Message:

PS3, Line 7: cluster
> s/clsuter/cluster/
Done


PS3, Line 9: 
> no semicolon?
Done


PS3, Line 10: 
> no semicolon?
Done


PS3, Line 12: erased
> s/earsed/erased/
Done


https://asterix-gerrit.ics.uci.edu/#/c/1549/3/asterixdb/asterix-server/src/main/opt/ansible/yaml/instance_stop.yml
File asterixdb/asterix-server/src/main/opt/ansible/yaml/instance_stop.yml:

PS3, Line 23: Drive
> Since we anyway need awk and awk does nice regex matching should we just us
Done


PS3, Line 31: `
> Here just using awk make things quite a bit more compact:
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib62483bfbb08ebd6f3ed97d1b64645541ce19d35
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <buyin...@gmail.com>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to