HIVE-29519: Redundant perfLogger reset and queryStartTime update (#6386)

Author: deniskuzZ

ql/src/java/org/apache/hadoop/hive/ql/Driver.java

@@ -170,8 +170,6 @@ private void runInternal(String command, boolean alreadyCompiled) throws Command
 
       if (!alreadyCompiled) {
         compileInternal(command, true);
-      } else {
-        driverContext.recordQueryStartTime();
       }
 
       DriverUtils.checkInterrupted(driverState, driverContext, "at acquiring the lock.", null, null);
@@ -273,14 +271,12 @@ private void validateCurrentSnapshot() throws CommandProcessorException {
           driverContext.setRetrial(true);
 
           compileInternal(context.getCmd(), true);
+          driverContext.setRetrial(false);
 
           if (driverContext.getPlan().hasAcidResourcesInQuery()) {
             driverTxnHandler.recordValidWriteIds();
             driverTxnHandler.setWriteIdForAcidFileSinks();
           }
-          // Since we're reusing the compiled plan, we need to update its start time for current run
-          driverContext.recordQueryStartTime();
-          driverContext.setRetrial(false);
         }
         // Re-check snapshot only in case we had to release locks and open a new transaction,
         // otherwise exclusive locks should protect output tables/partitions in snapshot from concurrent writes.

ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java

@@ -275,8 +275,4 @@ public void setExplainPlan(String explainPlan) {
   public String getExplainPlan() {
     return explainPlan;
   }
-
-  public void recordQueryStartTime() {
-    plan.setQueryStartTime(queryDisplay.getQueryStartTime());
-  }
 }