[ https://issues.apache.org/jira/browse/HIVE-18540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16340660#comment-16340660 ]
Song Jun commented on HIVE-18540: --------------------------------- [~anishek] Can you help to review this? Thanks > 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)