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