hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasanth Jayachandran (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (HIVE-14651) Add a local cluster for Tez and LLAP
Date Fri, 26 Aug 2016 22:47:20 GMT

    [ 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)

Mime
View raw message