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

Zoltan Haindrich commented on HIVE-14259:
-----------------------------------------

[~xuefuz] i've fixed another intresting subdirectory related method during 
HIVE-12244; that one i think matched '/backup/foo/bar' as a parent of '/foo' 
https://github.com/apache/hive/commit/47b5b5cdb8702c9ddbca3911b6a03ca59982b3c1#diff-c39d962bbd4b5ca645cfb9275b899a68L593

 [~lirui] the only "user" of this {{FileUtils::isSubDir}} method is {{Hive}}, 
it would be better to move this method there too? ...or...at first blink it 
seems like the only uses of these methods are originating from 
{{MoveTask}}...and these operations seem to be independent from the 
{{Hive::hiveDB}} threadlocal...

> FileUtils.isSubDir may return incorrect result
> ----------------------------------------------
>
>                 Key: HIVE-14259
>                 URL: https://issues.apache.org/jira/browse/HIVE-14259
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Zoltan Haindrich
>            Assignee: Zoltan Haindrich
>            Priority: Minor
>         Attachments: HIVE-14259.1.patch
>
>
> while I was working on HIVE-12244 i've looked around for utility 
> methods...i've found this method; but it considers path: `/dir12` inside 
> `/dir1`
> which is not true



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

Reply via email to