JingsongLi commented on a change in pull request #10401: [FLINK-15020][hive] 
Support timestamp type in hive
URL: https://github.com/apache/flink/pull/10401#discussion_r354100757
 
 

 ##########
 File path: 
flink-connectors/flink-connector-hive/src/main/java/org/apache/flink/connectors/hive/read/HiveTableInputFormat.java
 ##########
 @@ -170,8 +172,9 @@ private boolean useOrcVectorizedRead(HiveTablePartition 
partition) {
                        }
                }
 
-               LOG.info("Use flink orc ColumnarRow reader.");
-               return true;
+               // temporarily disable the vectorized reader, until we figure 
out how to read timestamps with it
 
 Review comment:
   No... I mean modify timestamp type, you can see above, there are support 
types.
   I think you can use https://github.com/apache/flink/pull/10426 to verify and 
wait a minute to merge.

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


With regards,
Apache Git Services

Reply via email to