[jira] [Issue Comment Deleted] (PHOENIX-3748) sqlline.py, sqlline-thin.py are using the wrong variables.

["Soonho Kim (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/PHOENIX-3748?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Soonho Kim updated PHOENIX-3748:
--------------------------------
    Comment: was deleted

(was: diff --git a/bin/sqlline-thin.py b/bin/sqlline-thin.py
index 47384d8..8ec39c6 100755
--- a/bin/sqlline-thin.py
+++ b/bin/sqlline-thin.py
@@ -113,7 +113,7 @@ if os.name == 'nt':
 
 # HBase configuration folder path (where hbase-site.xml reside) for
 # HBase/Phoenix client side property override
-hbase_config_path = os.getenv('HBASE_CONF_DIR', phoenix_utils.current_dir)
+hbase_config_path = os.getenv('HBASE_CONF_DIR', phoenix_utils.hbase_conf_dir)
 
 serialization = args.serialization if args.serialization else get_serialization()
 
diff --git a/bin/sqlline.py b/bin/sqlline.py
index 7a724de..3f89611 100755
--- a/bin/sqlline.py
+++ b/bin/sqlline.py
@@ -53,7 +53,7 @@ sqlfile = args.sqlfile
 
 # HBase configuration folder path (where hbase-site.xml reside) for
 # HBase/Phoenix client side property override
-hbase_config_path = os.getenv('HBASE_CONF_DIR', phoenix_utils.current_dir)
+hbase_config_path = os.getenv('HBASE_CONF_DIR', phoenix_utils.hbase_conf_dir)
 
 if sqlfile and not os.path.isfile(sqlfile):
     parser.print_help() )

> sqlline.py, sqlline-thin.py are using the wrong variables.
> ----------------------------------------------------------
>
>                 Key: PHOENIX-3748
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-3748
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 4.6.0, 4.8.0
>            Reporter: Soonho Kim
>            Priority: Trivial
>              Labels: patch
>
> It seems that the hbase_config_path variable in the sqlline.py and sqlline-thin.py scripts, are looking for invalid values.
> These variables should use the value "phoenix_utils.hbase_conf_dir" instead of "phoenix_utils.current_dir".
> -hbase_config_path = os.getenv('HBASE_CONF_DIR', phoenix_utils.current_dir)
> +hbase_config_path = os.getenv('HBASE_CONF_DIR', phoenix_utils.hbase_conf_dir)



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)