Remove DataHandle::commit()

From discussion with @leggett on !462 (merged), it seems like the DataHandle::commit() functionality is not used, and that using it is not envisioned anymore. I would thus tentatively propose to remove it.

Can you try this commit in your respective experiment frameworks and tell me how well it goes?

GaudiHive/src/AlgoExecutionTask.cpp

@@ -14,8 +14,8 @@
 tbb::task* AlgoExecutionTask::execute()
 {
 
-  IAlgorithm* ialg     = m_algorithm.get();
-  Algorithm* this_algo = dynamic_cast<Algorithm*>( ialg );
+  IAlgorithm* ialg      = m_algorithm.get();
+  Algorithm*  this_algo = dynamic_cast<Algorithm*>( ialg );
   if ( !this_algo ) {
     throw GaudiException( "Cast to Algorithm failed!", "AlgoExecutionTask", StatusCode::FAILURE );
   }
@@ -27,7 +27,7 @@ tbb::task* AlgoExecutionTask::execute()
   const SmartIF<IProperty> appmgr( m_serviceLocator );
 
   SmartIF<IMessageSvc> messageSvc( m_serviceLocator );
-  MsgStream log( messageSvc, "AlgoExecutionTask" );
+  MsgStream            log( messageSvc, "AlgoExecutionTask" );
 
   // select the appropriate store
   this_algo->whiteboard()->selectStore( m_evtCtx->valid() ? m_evtCtx->slot() : 0 ).ignore();
@@ -55,9 +55,6 @@ tbb::task* AlgoExecutionTask::execute()
     eventfailed = true;
   }
 
-  // Commit all DataHandles
-  this_algo->commitHandles();
-
   // DP it is important to propagate the failure of an event.
   // We need to stop execution when this happens so that execute run can
   // then receive the FAILURE