[ https://issues.apache.org/jira/browse/HIVE-26498?focusedWorklogId=806580&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-806580 ]
ASF GitHub Bot logged work on HIVE-26498: ----------------------------------------- Author: ASF GitHub Bot Created on: 07/Sep/22 04:31 Start Date: 07/Sep/22 04:31 Worklog Time Spent: 10m Work Description: kasakrisz commented on code in PR #3552: URL: https://github.com/apache/hive/pull/3552#discussion_r964371311 ########## iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergStorageHandler.java: ########## @@ -977,4 +977,11 @@ public Configuration get() { return conf; } } + + @Override + public String getCurrentSnapshot(org.apache.hadoop.hive.ql.metadata.Table hmsTable) { Review Comment: String gives us more flexibility: * I had a version when more than one field was returned from the iceberg snapshot serialized into json: snapshotId + sequenceNumber. Then it turned out that the snapshotId is enough so far. * Other than Iceberg storage formats may represents this value with another type and serialize to String is an option. Issue Time Tracking ------------------- Worklog Id: (was: 806580) Time Spent: 1.5h (was: 1h 20m) > Implement MV maintenance with Iceberg sources using full rebuild > ---------------------------------------------------------------- > > Key: HIVE-26498 > URL: https://issues.apache.org/jira/browse/HIVE-26498 > Project: Hive > Issue Type: Sub-task > Components: Materialized views > Reporter: Krisztian Kasa > Assignee: Krisztian Kasa > Priority: Major > Labels: pull-request-available > Time Spent: 1.5h > Remaining Estimate: 0h > > {code} > set hive.support.concurrency=true; > set hive.txn.manager=org.apache.hadoop.hive.ql.lockmgr.DbTxnManager; > create external table tbl_ice(a int, b string, c int) stored by iceberg > stored as orc tblproperties ('format-version'='2'); > insert into tbl_ice values (1, 'one', 50), (2, 'two', 51), (3, 'three', 52), > (4, 'four', 53), (5, 'five', 54); > create materialized view mat1 as > select b, c from tbl_ice where c > 52; > insert into tbl_ice values (111, 'one', 55), (333, 'two', 56); > explain cbo > alter materialized view mat1 rebuild; > alter materialized view mat1 rebuild; > {code} > MV full rebuild plan > {code} > CBO PLAN: > HiveProject(b=[$1], c=[$2]) > HiveFilter(condition=[>($2, 52)]) > HiveTableScan(table=[[default, tbl_ice]], table:alias=[tbl_ice]) > {code} -- This message was sent by Atlassian Jira (v8.20.10#820010)