[GitHub] [hive] pvargacl commented on a change in pull request #1866: HIVE-14165: Remove unnecessary file listing from FetchOperator

[GitBox <gi...@apache.org>]

pvargacl commented on a change in pull request #1866:
URL: https://github.com/apache/hive/pull/1866#discussion_r559448989



##########
File path: ql/src/test/queries/clientpositive/exim_04_evolved_parts.q
##########
@@ -15,7 +15,7 @@ alter table exim_employee_n12 clustered by (emp_sex, emp_dept) sorted by (emp_id
 alter table exim_employee_n12 add partition (emp_country='in', emp_state='tn');
 
 alter table exim_employee_n12 set fileformat 
-	inputformat  "org.apache.hadoop.hive.ql.io.BucketizedHiveInputFormat" 
+	inputformat  "org.apache.hadoop.hive.ql.io.RCFileInputFormat"

Review comment:
       BucketizedHiveInputFormat can not be used this way, it is an internal inputformat used by mapjoins and it requires someone to setup either has.map.work or has.reduce.work property.
   This test was passing because the partitions in this table are empty and the InputFormat.getSplits were not called for them in the FetchOperator. It is impossible to read from a table that has data in it and was setup like this.




----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org