HBASE-30033 Add tests for setFilter() batch validation

- testSetFilterWithBatchThrows: setBatch() then setFilter(PageFilter)
  should throw IncompatibleFilterException
- testSetFilterWithoutBatchDoesNotThrow: setFilter(PageFilter) without
  batch should succeed
- testSetFilterWithBatchAndNonFilterRowFilter: setBatch() then
  setFilter(FilterList) should succeed since FilterList.hasFilterRow()
  returns false

Author: mini666

hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestScan.java

@@ -29,6 +29,8 @@
 import org.apache.hadoop.hbase.HConstants;
 import org.apache.hadoop.hbase.client.Scan.ReadType;
 import org.apache.hadoop.hbase.filter.FilterList;
+import org.apache.hadoop.hbase.filter.IncompatibleFilterException;
+import org.apache.hadoop.hbase.filter.PageFilter;
 import org.apache.hadoop.hbase.security.access.Permission;
 import org.apache.hadoop.hbase.security.visibility.Authorizations;
 import org.apache.hadoop.hbase.testclassification.ClientTests;
@@ -254,6 +256,28 @@ public void testScanCopyConstructor() throws Exception {
       "Make sure copy constructor adds all the fields in the copied object");
   }
 
+  @Test(expected = IncompatibleFilterException.class)
+  public void testSetFilterWithBatchThrows() {
+    Scan scan = new Scan();
+    scan.setBatch(5);
+    scan.setFilter(new PageFilter(10));
+  }
+
+  @Test
+  public void testSetFilterWithoutBatchDoesNotThrow() {
+    Scan scan = new Scan();
+    scan.setFilter(new PageFilter(10));
+    // no exception expected
+  }
+
+  @Test
+  public void testSetFilterWithBatchAndNonFilterRowFilter() {
+    Scan scan = new Scan();
+    scan.setBatch(5);
+    scan.setFilter(new FilterList());
+    // FilterList.hasFilterRow() returns false, so no exception expected
+  }
+
   @Test
   public void testScanReadType() throws Exception {
     Scan scan = new Scan();