Sahil Takiar commented on HIVE-18672:

[~pvary] {{LogHelper#printInfo(String)}} will check the value of 
{{LogHelper#getIsSilent()}} before printing to the info stream. 
{{getIsSilent()}} checks the {{SessionState}} for its current value of 
{{isSilent}}. So I think it should be ok. We follow the same pattern (using 
{{LogHelper(LOG)}}) in {{SparkTask.java}}.

> Printed state in RemoteSparkJobMonitor is ambiguous
> ---------------------------------------------------
>                 Key: HIVE-18672
>                 URL: https://issues.apache.org/jira/browse/HIVE-18672
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Spark
>            Reporter: Sahil Takiar
>            Assignee: Sahil Takiar
>            Priority: Major
>         Attachments: HIVE-18672.1.patch, HIVE-18672.2.patch
> There are a few places in {{RemoteSparkJobMonitor}} (e.g. when the Spark job 
> is in state QUEUED) where the state of the Spark job is printed, but the info 
> is ambiguous (no reference to HoS, or the id of the Spark job).

This message was sent by Atlassian JIRA

Reply via email to