deniskuzZ commented on code in PR #5648:
URL: https://github.com/apache/hive/pull/5648#discussion_r1986355470
##########
ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveStorageHandler.java:
##########
@@ -285,6 +286,11 @@ default List<ColumnStatisticsObj>
getColStatistics(org.apache.hadoop.hive.ql.met
return null;
}
+ default AggrStats
getAggrColStatsFor(org.apache.hadoop.hive.ql.metadata.Table hmsTable,
List<String> colNames,
Review Comment:
done
##########
iceberg/iceberg-handler/src/test/java/org/apache/iceberg/mr/hive/HiveIcebergStorageHandlerWithEngineBase.java:
##########
@@ -108,57 +108,46 @@ public abstract class
HiveIcebergStorageHandlerWithEngineBase {
StatsSetupConst.TOTAL_SIZE, SnapshotSummary.TOTAL_FILE_SIZE_PROP
);
- @Parameters(name = "fileFormat={0}, engine={1}, catalog={2},
isVectorized={3}, formatVersion={4}")
+ @Parameters(name = "fileFormat={0}, catalog={1}, isVectorized={2},
formatVersion={3}")
public static Collection<Object[]> parameters() {
Collection<Object[]> testParams = Lists.newArrayList();
- String javaVersion = System.getProperty("java.specification.version");
// Run tests with every FileFormat for a single Catalog (HiveCatalog)
for (FileFormat fileFormat :
HiveIcebergStorageHandlerTestUtils.FILE_FORMATS) {
- for (String engine : EXECUTION_ENGINES) {
- IntStream.of(2, 1).forEach(formatVersion -> {
- // include Tez tests only for Java 8
- if (javaVersion.equals("1.8")) {
- testParams.add(new Object[]{fileFormat, engine,
TestTables.TestTableType.HIVE_CATALOG, false,
- formatVersion});
- // test for vectorization=ON in case of ORC and PARQUET format
with Tez engine
- if (fileFormat != FileFormat.METADATA && "tez".equals(engine) &&
HiveVersion.min(HiveVersion.HIVE_3)) {
- testParams.add(new Object[]{fileFormat, engine,
TestTables.TestTableType.HIVE_CATALOG, true,
- formatVersion});
- }
- }
- });
- }
+ IntStream.of(2, 1).forEach(formatVersion -> {
+ testParams.add(new Object[]{fileFormat, HIVE_CATALOG, false,
formatVersion});
+ // test for vectorization=ON in case of ORC and PARQUET format with
Tez engine
Review Comment:
done
--
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]
