Jim Apple has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16725 )

Change subject: IMPALA-10329 Change apt install retry times to 30
......................................................................


Patch Set 1:

> Hello, Jim, maybe we change the 'for' loop to 'while true'?

I see the logic in that. We can just get started when the system is ready!

But what happens if there's a problem other than this one that we can just wait 
out? Then the build would keep going for hours just failing at apt-get, yes?

Maybe we should look around for what the best practices are for this kind of 
use case. I think I may have seen something on askubuntu.com.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I790750da36ad53c87a830dfab6803a1862490daf
Gerrit-Change-Number: 16725
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <zhaoren...@hotmail.com>
Gerrit-Reviewer: Anonymous Coward <zhaoren...@hotmail.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbap...@apache.org>
Gerrit-Comment-Date: Tue, 17 Nov 2020 04:37:06 +0000
Gerrit-HasComments: No

Reply via email to