[jira] [Commented] (DRILL-6272) Remove binary jars files from source distribution

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

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

    https://github.com/apache/drill/pull/1225#discussion_r185373456
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestCTTAS.java ---
    @@ -35,41 +39,46 @@
     import org.apache.hadoop.fs.RemoteIterator;
     import org.apache.hadoop.fs.permission.FsPermission;
     import org.junit.BeforeClass;
    +import org.junit.Rule;
     import org.junit.Test;
     import org.junit.experimental.categories.Category;
    +import org.junit.rules.ExpectedException;
     
     import java.io.File;
     import java.io.IOException;
    +import java.nio.file.Files;
     import java.nio.file.Paths;
     import java.util.List;
    +import java.util.Optional;
     import java.util.Properties;
    -import java.util.UUID;
     
     import static org.apache.drill.exec.util.StoragePluginTestUtils.DFS_PLUGIN_NAME;
     import static org.apache.drill.exec.util.StoragePluginTestUtils.DFS_TMP_SCHEMA;
     import static org.hamcrest.CoreMatchers.containsString;
     import static org.junit.Assert.assertEquals;
    -import static org.junit.Assert.assertThat;
     import static org.junit.Assert.assertTrue;
     
     @Category(SqlTest.class)
     public class TestCTTAS extends BaseTestQuery {
     
    -  private static final UUID session_id = UUID.nameUUIDFromBytes("sessionId".getBytes());
       private static final String temp2_wk = "tmp2";
       private static final String temp2_schema = String.format("%s.%s", DFS_PLUGIN_NAME, temp2_wk);
     
    +  private static String sessionId;
       private static FileSystem fs;
       private static FsPermission expectedFolderPermission;
       private static FsPermission expectedFilePermission;
     
    +  @Rule
    +  public ExpectedException thrown = ExpectedException.none();
    +
       @BeforeClass
       public static void init() throws Exception {
    -    MockUp<UUID> uuidMockUp = mockRandomUUID(session_id);
    --- End diff --
    
     It is hard to say whether you want to change test logic or need a workaround for the removed `MockUp.tearDown()` method. In the latter case consider adding `getTemporaryName()` method to the `UserSession` class. The method can delegate to `UUID.randomUUID()` in production and can be mocked to return a predefined value in tests like this one. It will allow not to mock `UUID.randomUUID()` at all.


> Remove binary jars files from source distribution
> -------------------------------------------------
>
>                 Key: DRILL-6272
>                 URL: https://issues.apache.org/jira/browse/DRILL-6272
>             Project: Apache Drill
>          Issue Type: Task
>            Reporter: Vlad Rozov
>            Assignee: Arina Ielchiieva
>            Priority: Critical
>             Fix For: 1.14.0
>
>
> Per [~vrozov] the source distribution contains binary jar files under exec/java-exec/src/test/resources/jars



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)