You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Sahil Takiar (JIRA)" <ji...@apache.org> on 2018/05/31 17:09:00 UTC

[jira] [Commented] (HIVE-19602) Refactor inplace progress code in Hive-on-spark progress monitor to use ProgressMonitor instance

    [ https://issues.apache.org/jira/browse/HIVE-19602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16496867#comment-16496867 ] 

Sahil Takiar commented on HIVE-19602:
-------------------------------------

LGTM +1

[~lirui] might be interested in taking a look too, since he wrote most of this logic in HIVE-15039

> Refactor inplace progress code in Hive-on-spark progress monitor to use ProgressMonitor instance
> ------------------------------------------------------------------------------------------------
>
>                 Key: HIVE-19602
>                 URL: https://issues.apache.org/jira/browse/HIVE-19602
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Bharathkrishna Guruvayoor Murali
>            Assignee: Bharathkrishna Guruvayoor Murali
>            Priority: Major
>
> We can refactor the HOS inplace progress monitor code (SparkJobMonitor#printStatusInPlace) to use InplaceUpdate#render.
> We can create an instance of ProgressMonitor and use it to show the progress.
> This would be similar to :
> [https://github.com/apache/hive/blob/0b6bea89f74b607299ad944b37e4b62c711aaa69/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/RenderStrategy.java#L181]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)