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

ASF GitHub Bot commented on DRILL-5751:
---------------------------------------

Github user vvysotskyi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/927#discussion_r136373928
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/TestDynamicUDFSupport.java ---
    @@ -120,8 +132,11 @@ public void testDisableDynamicSupport() throws 
Exception {
       @Test
       public void testAbsentBinaryInStaging() throws Exception {
         Path staging = 
getDrillbitContext().getRemoteFunctionRegistry().getStagingArea();
    +    FileSystem fs = 
getDrillbitContext().getRemoteFunctionRegistry().getFs();
    +    copyJar(fs, jars, staging, default_binary_name);
    --- End diff --
    
    It seems that a source file should be copied here instead of the binary 
file, since test checks that binary file is missing.


> Fix unit tests to use local file system even if it is not set by default
> ------------------------------------------------------------------------
>
>                 Key: DRILL-5751
>                 URL: https://issues.apache.org/jira/browse/DRILL-5751
>             Project: Apache Drill
>          Issue Type: Bug
>    Affects Versions: 1.11.0
>            Reporter: Arina Ielchiieva
>            Assignee: Arina Ielchiieva
>             Fix For: 1.12.0
>
>
> When running test using, for example, mapr profile, some test fail with 
> FileNotFoundException.
> To fix the failures, we would need to modify related tests always to use 
> local file system, even if default file system is, for example, maprfs.
> List of affected unit tests:
> TestHashAggrSpill
> TestParquetWriter
> TestCTAS
> StorageStrategyTest
> TestDynamicUDFSupport
> etc.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to