ramitg254 commented on code in PR #6413:
URL: https://github.com/apache/hive/pull/6413#discussion_r3388289067
##########
ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java:
##########
@@ -3013,10 +3013,10 @@ private RelNode genTableLogicalPlan(String tableAlias,
QB qb) throws SemanticExc
ArrayList<ColumnInfo> partitionColumns = new ArrayList<ColumnInfo>();
// 3.2 Add column info corresponding to partition columns
- for (FieldSchema part_col : tabMetaData.getPartCols()) {
- colName = part_col.getName();
+ for (FieldSchema partCol : tabMetaData.getPartitionKeys()) {
Review Comment:
I tried enabling it earlier but it resulted in 200+ failures
https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-6453/48/tests/
haven't investigated failures yet as I am currently implementing concerns
suggested in other comments, but if you think we should enable it as a part of
this pr only and not some separate follow up or already opened jira which can
take care of it, then I can check it after resolving other comments
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
