Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15168 )
Change subject: [util] remove timeout parameter for cloud::InstanceDetector ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15168/1/src/kudu/util/cloud/instance_detector.cc File src/kudu/util/cloud/instance_detector.cc: http://gerrit.cloudera.org:8080/#/c/15168/1/src/kudu/util/cloud/instance_detector.cc@65 PS1, Line 65: // Add an extra delay in addition to the declared timeout for fetching : // instance metadata. The idea is to have the limit on the time to wait : // in case if instance metadata fetcher could not fulfill its promise to : // return valid result or an error within the init_timeout() interval. > Good point; I missed that the destructor joins on the threads. Thank you for the feedback! I was hesitant to use Wait() instead of WaitUntil(), but now I understand it was just paranoia. I'm going to get rid of these ugly timeouts. -- To view, visit http://gerrit.cloudera.org:8080/15168 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6f7cac9273e884abda22b9e9a7d37fa3e307c2d4 Gerrit-Change-Number: 15168 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Thu, 06 Feb 2020 02:34:39 +0000 Gerrit-HasComments: Yes
