You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Prasanth Jayachandran (JIRA)" <ji...@apache.org> on 2016/08/26 22:47:20 UTC

[jira] [Comment Edited] (HIVE-14651) Add a local cluster for Tez and LLAP

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

Prasanth Jayachandran edited comment on HIVE-14651 at 8/26/16 10:46 PM:
------------------------------------------------------------------------

nit: LOG.error instead?

{code}
  try {
      fs.mkdirs(userInstallPath);
   } catch (IOException e) {
      LOG.info("Failed to create path={}. Continuing. Exception message={}", warehousePath,
      e.getMessage());
   }
{code}

There are 2 other places as well.

Other than that lgtm, +1 



was (Author: prasanth_j):
nit: LOG.error instead?

{code}
  try {
      fs.mkdirs(userInstallPath);
   } catch (IOException e) {
      LOG.info("Failed to create path={}. Continuing. Exception message={}", warehousePath,
      e.getMessage());
   }
{code}

There are 2 other places as well.

Other that that lgtm, +1 


> Add a local cluster for Tez and LLAP
> ------------------------------------
>
>                 Key: HIVE-14651
>                 URL: https://issues.apache.org/jira/browse/HIVE-14651
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Testing Infrastructure
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: HIVE-14651.01.patch, HIVE-14651.02.patch, HIVE-14651.03.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)