paul8263 commented on code in PR #10517:
URL: https://github.com/apache/hudi/pull/10517#discussion_r1458202085


##########
hudi-flink-datasource/hudi-flink/src/main/java/org/apache/hudi/source/ExpressionPredicates.java:
##########
@@ -602,10 +602,10 @@ private static FilterPredicate 
toParquetPredicate(FunctionDefinition functionDef
       case TINYINT:
       case SMALLINT:
       case INTEGER:
+      case DATE:
       case TIME_WITHOUT_TIME_ZONE:
         return predicateSupportsLtGt(functionDefinition, 
intColumn(columnName), (Integer) literal);
       case BIGINT:
-      case DATE:
       case TIMESTAMP_WITHOUT_TIME_ZONE:

Review Comment:
   Hi @danny0405 ,
   
   Please see: [HUDI-7303](https://issues.apache.org/jira/browse/HUDI-7303).
   
   In parquet, date type is stored as INT32 (epoch day). But if we add some 
conditions with date typed field in SQL where clause, its type will 
unexpectedly convert to Long.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to