[ https://issues.apache.org/jira/browse/HIVE-2864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Phabricator updated HIVE-2864: ------------------------------ Attachment: HIVE-2864.D2265.1.patch kevinwilfong requested code review of "HIVE-2864 [jira] If hive history file's directory doesn't exist don't crash". Reviewers: JIRA https://issues.apache.org/jira/browse/HIVE-2864 Displays a warning to the user if the directory for the history file does not exist, rather than crashing the session. Currently, if the history file's directory does not exist the Hive client crashes. Instead, since this is not a vital feature, it should just display a warning to the user and continue without it. This will become more important once the directory becomes configurable, see: https://issues.apache.org/jira/browse/HIVE-1708 TEST PLAN EMPTY REVISION DETAIL https://reviews.facebook.net/D2265 AFFECTED FILES cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java MANAGE HERALD DIFFERENTIAL RULES https://reviews.facebook.net/herald/view/differential/ WHY DID I GET THIS EMAIL? https://reviews.facebook.net/herald/transcript/5013/ Tip: use the X-Herald-Rules header to filter Herald messages in your client. > If hive history file's directory doesn't exist don't crash > ---------------------------------------------------------- > > Key: HIVE-2864 > URL: https://issues.apache.org/jira/browse/HIVE-2864 > Project: Hive > Issue Type: Improvement > Reporter: Kevin Wilfong > Assignee: Kevin Wilfong > Attachments: HIVE-2864.D2265.1.patch > > > Currently, if the history file's directory does not exist the Hive client > crashes. Instead, since this is not a vital feature, it should just display > a warning to the user and continue without it. > This will become more important once the directory becomes configurable, see: > https://issues.apache.org/jira/browse/HIVE-1708 -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira