[GitHub] [incubator-iceberg] aokolnychyi commented on issue #327: Introduce startsWith Predicate

[GitBox <gi...@apache.org>]

aokolnychyi commented on issue #327: Introduce startsWith Predicate
URL: https://github.com/apache/incubator-iceberg/pull/327#issuecomment-523089442
 
 
   I would like to follow up with a PR to enable pushdown of `startsWith` in Spark. Are there any limitations that will block me?
   
   Apart from changes in `SparkFilters`, we will need to extend `InclusiveMetricsEvaluator`, `ParquetDictionaryRowGroupFilter`, `ParquetMetricsRowGroupFilter` with logic that checks lower/upper bounds in `startsWith`.
   
   I think one way to implement this is to take the same number of bytes from prefix and bounds (e.g. slice) and then rely on `UnsignedByteBufComparator` for unsigned lexicographical comparison. Alternatively, we can convert bounds to strings and operate on them.
   
   @sujithjay @rdblue @moulimukherjee Any thoughts?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org