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

Song Jun edited comment on HIVE-18540 at 2/2/18 10:08 AM:
----------------------------------------------------------

[~anishek] thanks for your reply.

I use iTerm2, usually I open serval session window in one Tab, so the screen is 
divided, and the width of one of session window may be very small, if I close 
others, the width will become larger, and it should show the normally format 
output .

 

I think it will better to show the output or show some logs to warn the user 
that the width is not enough


was (Author: windpiger):
[~anishek] thanks for your reply.

I use iTerm2, usually I open serval session window in one Tab, so the screen is 
divided, and the width of one of session window may be very small, if I close 
others, the width will become larger, and it will show the normally format 
output .

 

I think it will better to show the output or show some logs to warn the user 
that the width is not enough

>  remove logic for wide terminal to display in-place updates
> -----------------------------------------------------------
>
>                 Key: HIVE-18540
>                 URL: https://issues.apache.org/jira/browse/HIVE-18540
>             Project: Hive
>          Issue Type: Improvement
>    Affects Versions: 2.3.2
>            Reporter: Song Jun
>            Assignee: Song Jun
>            Priority: Minor
>         Attachments: HIVE-18540.patch
>
>
> Hive Shell/beeline run Tez job, when the terminal wide is lower than 95, 
> there will be no output(progress bar) to the console, because of the 
> following limit logic in class InPlaceUpdate:
> {code:java}
> public static boolean canRenderInPlace(HiveConf conf) { boolean 
> inPlaceUpdates = HiveConf.getBoolVar(conf, 
> HiveConf.ConfVars.TEZ_EXEC_INPLACE_PROGRESS); // we need at least 80 chars 
> wide terminal to display in-place updates properly return inPlaceUpdates && 
> isUnixTerminal() && TerminalFactory.get().getWidth() >= MIN_TERMINAL_WIDTH; 
> }{code}
>  
> {code:java}
> && isUnixTerminal() && TerminalFactory.get().getWidth() >= 
> MIN_TERMINAL_WIDTH;{code}
> If someone do not know this logic ,and the terminal is not enough wide, he 
> will think something wrong and do know how to fix it.
>  
> and I think it is better to remove the trouble logic.
>  



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

Reply via email to