Mailing-List: contact dev-help@hive.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hive.apache.org
Content-Type: multipart/alternative;
 boundary="===============2418705070618742101=="
MIME-Version: 1.0
Subject: Review Request 49984: HIVE-14215: Displaying inconsistent CPU usage
 data with MR execution engine
From: Peter Vary <pvary@cloudera.com>
To: Sergio Pena <sergio.pena@cloudera.com>, Aihua Xu <axu@cloudera.com>
Cc: Peter Vary <pvary@cloudera.com>, hive <dev@hive.apache.org>
Date: Wed, 13 Jul 2016 10:06:15 -0000
Message-ID: <20160713100615.8919.35710@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: Peter Vary <noreply@reviews.apache.org>
Reply-To: Peter Vary <pvary@cloudera.com>
archived-at: Wed, 13 Jul 2016 10:06:19 -0000

--===============2418705070618742101==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49984/
-----------------------------------------------------------

Review request for hive, Aihua Xu and Sergio Pena.


Bugs: HIVE-14215
    https://issues.apache.org/jira/browse/HIVE-14215


Repository: hive-git


Description
-------

Reordered the time caluclation, so the displayed data is consistent now


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/HadoopJobExecHelper.java cfb4a28 

Diff: https://reviews.apache.org/r/49984/diff/


Testing
-------

Manually.

Should have to put sleep at the end of the while cycle, to consistently reproduce the problem.
Keeping the same sleep there, the displayed data is consistent with the patch.

Sleep removed from the code in the end :)

If you have any idea, how to produce a reliable test case for this, I am open for every suggestions.

Thanks,
Peter


Thanks,

Peter Vary


--===============2418705070618742101==--