This is an automated email from the ASF dual-hosted git repository.

wesm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new efa28a7  ARROW-3393: [C++] Add missing override on virtual dtor in 
task-group.cc
efa28a7 is described below

commit efa28a7cc32d345468a8cf4d8c953c0d1d812ab9
Author: Wes McKinney <[email protected]>
AuthorDate: Mon Oct 1 17:15:04 2018 -0400

    ARROW-3393: [C++] Add missing override on virtual dtor in task-group.cc
    
    Author: Wes McKinney <[email protected]>
    
    Closes #2678 from wesm/ARROW-3393 and squashes the following commits:
    
    5fc19e153 <Wes McKinney> Add missing override on virtual dtor
---
 cpp/src/arrow/util/task-group.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/util/task-group.cc b/cpp/src/arrow/util/task-group.cc
index f4b0664..dd551fa 100644
--- a/cpp/src/arrow/util/task-group.cc
+++ b/cpp/src/arrow/util/task-group.cc
@@ -69,7 +69,7 @@ class ThreadedTaskGroup : public TaskGroup {
  public:
   explicit ThreadedTaskGroup(ThreadPool* thread_pool) : 
thread_pool_(thread_pool) {}
 
-  ~ThreadedTaskGroup() {
+  ~ThreadedTaskGroup() override {
     // Make sure all pending tasks are finished, so that dangling references
     // to this don't persist.
     ARROW_UNUSED(Finish());

Reply via email to