Indhumathi27 commented on a change in pull request #3913:
URL: https://github.com/apache/carbondata/pull/3913#discussion_r484849645
##########
File path:
integration/presto/src/main/prestosql/org/apache/carbondata/presto/impl/CarbonTableReader.java
##########
@@ -245,16 +242,14 @@ private CarbonTableCacheModel
getValidCacheBySchemaTableName(SchemaTableName sch
*
* @param tableCacheModel cached table
* @param filters carbonData filters
- * @param constraints presto filters
+ * @param filteredPartitions matched partitionSpec for the filter
* @param config hadoop conf
* @return list of multiblock split
* @throws IOException
*/
- public List<CarbonLocalMultiBlockSplit> getInputSplits(
- CarbonTableCacheModel tableCacheModel,
- Expression filters,
- TupleDomain<HiveColumnHandle> constraints,
- Configuration config) throws IOException {
+ public List<CarbonLocalMultiBlockSplit> getInputSplits(CarbonTableCacheModel
tableCacheModel,
+ Expression filters, List<PartitionSpec> filteredPartitions,
Configuration config)
Review comment:
Can revert to old style
----------------------------------------------------------------
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:
[email protected]
