----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61663/#review184121 -----------------------------------------------------------
Patch looks good. I didn't read the js files, but hope it's okay. a few minor comments. ql/src/java/org/apache/hadoop/hive/ql/QueryDisplay.java Lines 127 (patched) <https://reviews.apache.org/r/61663/#comment260180> What happens if this is a map-only task? ql/src/java/org/apache/hadoop/hive/ql/QueryDisplay.java Lines 132 (patched) <https://reviews.apache.org/r/61663/#comment260181> It might be better if the null check is put in getCountersJson() method. service/src/resources/hive-webapps/static/css/query-plan-graph.css Lines 1 (patched) <https://reviews.apache.org/r/61663/#comment260183> Apache license header if possible. service/src/resources/hive-webapps/static/js/query-plan-graph.js Lines 1 (patched) <https://reviews.apache.org/r/61663/#comment260182> I think we need apache license header. service/src/resources/hive-webapps/static/js/vis.min.js Lines 1 (patched) <https://reviews.apache.org/r/61663/#comment260184> Apache license header. - Xuefu Zhang On Aug. 16, 2017, 1:55 p.m., Karen Coppage wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61663/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2017, 1:55 p.m.) > > > Review request for hive, Peter Vary and Xuefu Zhang. > > > Repository: hive-git > > > Description > ------- > > I’m working on a feature of the Hive WebUI Query Plan tab that would provide > the option to display the query plan as a nice graph (scroll down for > screenshots). If you click on one of the graph’s stages, the plan for that > stage appears as text below. > Stages are color-coded if they have a status (Success, Error, Running), and > the rest are grayed out. Coloring is based on status already available in the > WebUI, under the Stages tab. > There is an additional option to display stats for MapReduce tasks. This > includes the job’s ID, tracking URL (where the logs are found), and mapper > and reducer numbers/progress, among other info. > The library I’m using for the graph is called vis.js (http://visjs.org/). It > has an Apache license, and the only necessary file to be included from this > library is about 700 KB. > I tried to keep server-side changes minimal, and graph generation is taken > care of by the client. Plans with more than a given number of stages > (default: 25) won't be displayed in order to preserve resources. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/common/LogUtils.java 0a3e0c7201 > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 3c158a6692 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java 4e7c80f184 > ql/src/java/org/apache/hadoop/hive/ql/MapRedStats.java 4b6051485e > ql/src/java/org/apache/hadoop/hive/ql/QueryDisplay.java bf6cb91745 > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/HadoopJobExecHelper.java > 3c0719717c > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapRedTask.java 1bd4db7805 > service/src/jamon/org/apache/hive/tmpl/QueryProfileTmpl.jamon ff7476ee02 > service/src/resources/hive-webapps/static/css/query-plan-graph.css > PRE-CREATION > service/src/resources/hive-webapps/static/js/query-plan-graph.js > PRE-CREATION > service/src/resources/hive-webapps/static/js/vis.min.js PRE-CREATION > > > Diff: https://reviews.apache.org/r/61663/diff/3/ > > > Testing > ------- > > > Thanks, > > Karen Coppage > >