drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From arina-ielchiieva <...@git.apache.org>
Subject [GitHub] drill pull request #1225: DRILL-6272: Refactor dynamic UDFs and function ini...
Date Wed, 02 May 2018 11:22:40 GMT
Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1225#discussion_r185452673
  
    --- 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 --
    
    I had to re-write this part since `MockUp.tearDown()` is not present in newer jmockit
version. Now we don't mock session id at all.


---

Mime
View raw message