[GitHub] incubator-omid pull request #41: [OMID-102] Support for user Filter when usi...

[JamesRTaylor]

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

    https://github.com/apache/incubator-omid/pull/41#discussion_r207919955
  
    --- Diff: 
hbase-coprocessor/src/test/java/org/apache/omid/transaction/TestSnapshotFilter.java
 ---
    @@ -226,8 +222,115 @@ public void testGetFirstResult() throws Throwable {
         }
     
         @Test(timeOut = 60_000)
    -    public void testGetSecondResult() throws Throwable {
    --- End diff --
    
    Is there a test which would have failed without this patch (i.e. one that 
demonstrates the need for having the visibility filtering done as a pure HBase 
filter)?


---