vineetgarg02 commented on a change in pull request #552: Hive 21279
URL: https://github.com/apache/hive/pull/552#discussion_r260890741
 
 

 ##########
 File path: ql/src/java/org/apache/hadoop/hive/ql/exec/FetchOperator.java
 ##########
 @@ -379,6 +387,11 @@ public boolean doNext(WritableComparable key, Writable 
value) throws IOException
       Class<? extends InputFormat> formatter = 
currDesc.getInputFileFormatClass();
       Utilities.copyTableJobPropertiesToConf(currDesc.getTableDesc(), job);
       InputFormat inputFormat = getInputFormatFromCache(formatter, job);
+      if(inputFormat instanceof HiveSequenceFileInputFormat) {
+        // input format could be cached, in which case we need to reset the 
list of files to fetch
 
 Review comment:
   @ashutoshc I don't think result cache caches input format. This caching has 
been there way before result cache so I don't think this comment will be 
correct.

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


With regards,
Apache Git Services

Reply via email to