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 3: > And I think this solution is same with 'changeing for loop to while > true', just different checking lock solution. Sorry, I missed this part of your message. I find the two solutions different, as apt-get can fail for reasons other than another process holding the lock. This solution reduces the number of scenarios in which a build spins indefinitely, which is a result that should be avoided in terms of CI/CD ergonomics and economics. -- 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: 3 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Comment-Date: Thu, 19 Nov 2020 04:47:59 +0000 Gerrit-HasComments: No
