You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by "Sumit Mohanty (JIRA)" <ji...@apache.org> on 2015/05/21 22:24:19 UTC

[jira] [Comment Edited] (AMBARI-11312) Use default value of hbase.tmp.dir instead of hard coded directory

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

Sumit Mohanty edited comment on AMBARI-11312 at 5/21/15 8:23 PM:
-----------------------------------------------------------------

We need more changes.

ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py

see lines
{code}
tmp_dir = config['configurations']['hbase-site']['hbase.tmp.dir']
{code}

ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/params.py

{code}
base_tmp_dir = config['configurations']['hbase-site']['hbase.tmp.dir']
{code}

we need to use default syntax and check for None for both. Can you look for all usage of that variable and make sure it can handle the case when it is not specified.



was (Author: sumitmohanty):
LGTM, +1

> Use default value of hbase.tmp.dir instead of hard coded directory
> ------------------------------------------------------------------
>
>                 Key: AMBARI-11312
>                 URL: https://issues.apache.org/jira/browse/AMBARI-11312
>             Project: Ambari
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: ambari-11312-v1.txt
>
>
> Currently hbase.tmp.dir is hard coded with /hadoop/hbase .
> We saw the following error when testing Phoenix UDF functionality:
> {code}
> 15/05/17 14:56:11 WARN util.DynamicClassLoader: Failed to load new jar UDF_artifact-1.0-SNAPSHOT.jar
> java.io.FileNotFoundException: /grid/0/var/log/hbase/local/jars/UDF_artifact-1.0-SNAPSHOT.jar (Permission denied)
> 	at java.io.FileOutputStream.open(Native Method)
> 	at java.io.FileOutputStream.<init>(FileOutputStream.java:221)
> 	at org.apache.hadoop.fs.RawLocalFileSystem$LocalFSFileOutputStream.<init>(RawLocalFileSystem.java:222)
> 	at org.apache.hadoop.fs.RawLocalFileSystem$LocalFSFileOutputStream.<init>(RawLocalFileSystem.java:209)
> 	at org.apache.hadoop.fs.RawLocalFileSystem.createOutputStreamWithMode(RawLocalFileSystem.java:305)
> 	at org.apache.hadoop.fs.RawLocalFileSystem.create(RawLocalFileSystem.java:293)
> 	at org.apache.hadoop.fs.RawLocalFileSystem.create(RawLocalFileSystem.java:326)
> {code}
> Default value for hbase.tmp.dir should be used instead.



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