Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12062
Change subject: IMPALA-7946: Use original timeout in THREAD_POOL_TASK_TIMED_OUT message ...................................................................... IMPALA-7946: Use original timeout in THREAD_POOL_TASK_TIMED_OUT message When SynchronousThreadPool::SynchronousOffer() times out, it can sometimes print the wrong time out in the error message. This happens because it is enforcing a total timeout across multiple operations. For example, if there is a total timeout of 5 seconds and the first step takes 1 second, the remaining step is given a 4 second timeout to enforce the total timeout. However, this 4 second timeout should not be expressed in the THREAD_POOL_TASK_TIMED_OUT error message if the task times out. Instead, SynchronousOffer() should always use the original timeout as the internal time out is unimportant to users. This changes the code to make the error message always use the original timeout. Change-Id: Ib7bc31f58a8d29abfdc24959dc2730a0ae24ec56 --- M be/src/util/thread-pool.h 1 file changed, 15 insertions(+), 13 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/62/12062/1 -- To view, visit http://gerrit.cloudera.org:8080/12062 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib7bc31f58a8d29abfdc24959dc2730a0ae24ec56 Gerrit-Change-Number: 12062 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>
