> On Nov. 16, 2017, 7:45 p.m., Prasanth_J wrote:
> > llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java
> > Lines 414 (patched)
> > <https://reviews.apache.org/r/63864/diff/1/?file=1894037#file1894037line414>
> >
> >     or update log4j2 properties file with WARN default for WM_LOG logger.
> 
> Sergey Shelukhin wrote:
>     Well, for now I want them to always be logged

You could add the logger at INFO level in properties file and switch to WARN 
when you don't need it. Changing the source requires recompilation on every 
change.


- Prasanth_J


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


On Nov. 16, 2017, 3:36 a.m., Sergey Shelukhin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63864/
> -----------------------------------------------------------
> 
> (Updated Nov. 16, 2017, 3:36 a.m.)
> 
> 
> Review request for hive and Prasanth_J.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   
> llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskCommunicator.java
>  a02a414a76 
>   
> llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java
>  9dc521e39e 
>   
> ql/src/java/org/apache/hadoop/hive/ql/exec/tez/GuaranteedTasksAllocator.java 
> a1775cd6bb 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/UserPoolMapping.java 
> 851245c154 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java 
> 1fe5859490 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java 
> 5ba6639e0c 
> 
> 
> Diff: https://reviews.apache.org/r/63864/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Shelukhin
> 
>

Reply via email to