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

Hari Sankar Sivarama Subramaniyan commented on HIVE-10684:
----------------------------------------------------------

[~Ferd]  Sorry for the delay, I have a few minor comments here.   
1. For public int executeCmd (), can you make this a private function. Also the 
return value from this function is not used; i think it should either be 
removed or have it logged for debugging purpose. 
2. 
{code}
+//      Files.copy(new File("/tmp/" + clazzV2FileName.toString()), dist);
{code}

The above line can be removed.

Thanks
Hari

> Fix the unit test failures for HIVE-7553 after HIVE-10674 removed the binary 
> jar files
> --------------------------------------------------------------------------------------
>
>                 Key: HIVE-10684
>                 URL: https://issues.apache.org/jira/browse/HIVE-10684
>             Project: Hive
>          Issue Type: Bug
>          Components: Tests
>            Reporter: Ferdinand Xu
>            Assignee: Ferdinand Xu
>         Attachments: HIVE-10684.1.patch, HIVE-10684.patch
>
>




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

Reply via email to