> On Oct. 22, 2014, 11:45 a.m., Jinfeng Ni wrote:
> > contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/HBaseFilterBuilder.java,
> >  line 228
> > <https://reviews.apache.org/r/27026/diff/1/?file=728661#file728661line228>
> >
> >     There is a utility function which will convert from SQL like pattern 
> > into java regex pattern. You may use that driectly.
> >     
> >     org.apache.drill.exec.expr.fn.impl.RegexpUtil.sqlToRegexLike 
> >     
> >     Also, LIKE operator could have ESCAPE. You may need consider the ESCAPE 
> > character (The utility method allows to pass the ESCAPE character).

Thanks for the pointer. I looked at the utility method and think it does no 
quite meet the need since I also need to extract the prefix, if any, so that I 
can alter shrink the scan range.


- Aditya


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27026/#review57855
-----------------------------------------------------------


On Oct. 22, 2014, 12:34 a.m., Aditya Kishore wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27026/
> -----------------------------------------------------------
> 
> (Updated Oct. 22, 2014, 12:34 a.m.)
> 
> 
> Review request for drill, Jacques Nadeau and Steven Phillips.
> 
> 
> Bugs: DRILL-1508
>     https://issues.apache.org/jira/browse/DRILL-1508
> 
> 
> Repository: drill-git
> 
> 
> Description
> -------
> 
> The attached patch translate a LIKE operator in a WHERE clause into a 
> `RegexStringComparator()` when applicable.
> 
> 
> Diffs
> -----
> 
>   
> contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/CompareFunctionsProcessor.java
>  6810f81 
>   
> contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/HBaseFilterBuilder.java
>  ee011a5 
>   
> contrib/storage-hbase/src/test/java/org/apache/drill/hbase/HBaseTestsSuite.java
>  18cf87c 
>   
> contrib/storage-hbase/src/test/java/org/apache/drill/hbase/TestHBaseFilterPushDown.java
>  29e7033 
>   
> contrib/storage-hbase/src/test/java/org/apache/drill/hbase/TestTableGenerator.java
>  ccf5ad5 
> 
> Diff: https://reviews.apache.org/r/27026/diff/
> 
> 
> Testing
> -------
> 
> Added two new unit test cases `testFilterPushDownRowKeyLike()` and 
> `testFilterPushDownRowKeyRangeAndColumnValueLike()`.
> 
> 
> Thanks,
> 
> Aditya Kishore
> 
>

Reply via email to