[ https://issues.apache.org/jira/browse/HIVE-27187?focusedWorklogId=860991&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-860991 ]
ASF GitHub Bot logged work on HIVE-27187: ----------------------------------------- Author: ASF GitHub Bot Created on: 08/May/23 11:04 Start Date: 08/May/23 11:04 Worklog Time Spent: 10m Work Description: deniskuzZ commented on code in PR #4278: URL: https://github.com/apache/hive/pull/4278#discussion_r1187315293 ########## ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java: ########## @@ -3096,6 +3096,8 @@ Seems much cleaner if each stmt is identified as a particular HiveOperation (whi assert t != null; if (AcidUtils.isTransactionalTable(t) && sharedWrite) { compBuilder.setSharedWrite(); + } else if (MetaStoreUtils.isNonNativeTable(t.getTTable())) { + compBuilder.setLock(getLockTypeFromStorageHandler(output, t)); Review Comment: 👍 Issue Time Tracking ------------------- Worklog Id: (was: 860991) Time Spent: 5.5h (was: 5h 20m) > Incremental rebuild of materialized view having aggregate and stored by > iceberg > ------------------------------------------------------------------------------- > > Key: HIVE-27187 > URL: https://issues.apache.org/jira/browse/HIVE-27187 > Project: Hive > Issue Type: Improvement > Components: Iceberg integration, Materialized views > Reporter: Krisztian Kasa > Assignee: Krisztian Kasa > Priority: Major > Labels: pull-request-available > Time Spent: 5.5h > Remaining Estimate: 0h > > Currently incremental rebuild of materialized view stored by iceberg which > definition query contains aggregate operator is transformed to an insert > overwrite statement which contains a union operator if the source tables > contains insert operations only. One branch of the union scans the view the > other produces the delta. > This can be improved further: transform the statement to a multi insert > statement representing a merge statement to insert new aggregations and > update existing. -- This message was sent by Atlassian Jira (v8.20.10#820010)