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:
[email protected]
With regards,
Apache Git Services
