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

Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2019/04/01 16:15:08 UTC

[native-toolchain-CR] Parallelize image building process

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12864 )

Change subject: Parallelize image building process
......................................................................


Patch Set 3: Verified+1 Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12864/3/docker/buildall.py
File docker/buildall.py:

http://gerrit.cloudera.org:8080/#/c/12864/3/docker/buildall.py@49
PS3, Line 49:   sys.exit(exit)
I didn't realise you could pass strings to sys.exit() so I looked at the docs and learned something new.



-- 
To view, visit http://gerrit.cloudera.org:8080/12864
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I526b0088020bb7c647f1918b3eb3258cc1e3ddf9
Gerrit-Change-Number: 12864
Gerrit-PatchSet: 3
Gerrit-Owner: Hector Acosta <he...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 01 Apr 2019 16:15:08 +0000
Gerrit-HasComments: Yes