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

Dong Chen commented on HIVE-8138:
---------------------------------

Hi [~brocknoland], the patch looks good. Thanks for making the global init file 
better.

Just one small comment about code style.
Can we add a newline between the first two test case in 
TestSessionGlobalInitFile.java?

BTW, to rebase the patch, I remember a Jira has renamed the property 
"HIVE_GLOBAL_INIT_FILE_LOCATION" to "HIVE_SERVER2_GLOBAL_..." for consistency. 
Maybe renaming the property in this patch will fix the merging confilct.

> Global Init file should allow specifying file name  not only directory
> ----------------------------------------------------------------------
>
>                 Key: HIVE-8138
>                 URL: https://issues.apache.org/jira/browse/HIVE-8138
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Brock Noland
>            Assignee: Brock Noland
>         Attachments: HIVE-8138.patch
>
>
> HIVE-5160 allows you to specify a directory where a .hiverc file exists. 
> However since .hiverc is a hidden file this can be confusing. The property 
> should allow a path to a file or a directory.



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

Reply via email to