Wrong multiple counting of resultSetTimeMs and wallclockTimeMs in OverallQueryMetrics

We update wallClockTimeMS and resultSetTimeMS when we reach the last row in our ResultSet iterations (see this). However, this is also called inside PhoenixResultSet.close(), see this.

The problem is, these calls in-turn add elapsedTime again (endQuery and stopResultsetWatch) and so these metrics get 2X the value they should. Ideally we should change the metrics only if the stopWatch was running before calling stopWatch.stop or else we will count it multiple times.