deniskuzZ commented on code in PR #6413:
URL: https://github.com/apache/hive/pull/6413#discussion_r3396271389
##########
ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnStatsSemanticAnalyzer.java:
##########
@@ -107,7 +107,8 @@ private boolean shouldRewrite(ASTNode tree) {
*/
private static FieldSchemas getStatsEligibleFieldSchemas(Table tbl) {
List<FieldSchema> result = new ArrayList<>();
- for (FieldSchema col : tbl.getCols()) {
+ List<FieldSchema> colsToLookUp = tbl.hasNonNativePartitionSupport() ?
tbl.getAllCols() : tbl.getCols();
Review Comment:
in case of native tables we do not include partition columns into partition
stats - it doesn't make sense since those = identity partition value.
iceberg has transforms, so maybe it makes sence
--
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]
