Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/996#discussion_r146818260
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/store/dfs/TestFileSelection.java
 ---
    @@ -63,4 +63,17 @@ public void testEmptyFolderThrowsTableNotFound() throws 
Exception {
         }
       }
     
    --- End diff --
    
    Maybe it makes sense to move new unit tests and 
`testEmptyFolderThrowsTableNotFound` into separate class which will test 
behavior when incorrect object is defined (i.e. schema, workspace, table)?


---

Reply via email to