marton-bod commented on code in PR #3236:
URL: https://github.com/apache/hive/pull/3236#discussion_r856213274
##########
iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergStorageHandler.java:
##########
@@ -744,8 +745,17 @@ private String
collectColumnAndReplaceDummyValues(ExprNodeDesc node, String foun
return column;
}
- private void fallbackToNonVectorizedModeForV2(Properties tableProps) {
- if ("2".equals(tableProps.get(TableProperties.FORMAT_VERSION))) {
+ /**
+ * If any of the following checks is true we fall back to non vectorized
mode:
+ * <ul>
+ * <li>iceberg format-version is "2"</li>
+ * <li>fileformat is set to avro</li>
+ * </ul>
+ * @param tableProps table properties, must be not null
+ */
+ private void fallbackToNonVectorizedModeBasedOnProperties(Properties
tableProps) {
+ if ("2".equals(tableProps.get(TableProperties.FORMAT_VERSION)) ||
+ FileFormat.AVRO.name().equalsIgnoreCase((String)
tableProps.get(TableProperties.DEFAULT_FILE_FORMAT))) {
Review Comment:
nit: you can use `tablePros.getProperty()` instead of casting
--
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]
