[
https://issues.apache.org/jira/browse/HIVE-26228?focusedWorklogId=774439&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-774439
]
ASF GitHub Bot logged work on HIVE-26228:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 25/May/22 09:11
Start Date: 25/May/22 09:11
Worklog Time Spent: 10m
Work Description: pvary commented on code in PR #3287:
URL: https://github.com/apache/hive/pull/3287#discussion_r881416218
##########
ql/src/java/org/apache/hadoop/hive/ql/parse/AlterTableExecuteSpec.java:
##########
@@ -0,0 +1,90 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.hadoop.hive.ql.parse;
+
+/**
+ * Execute operation specification. It stores the type of the operation and
its parameters.
+ * The following operation are supported
+ * <ul>
+ * <li>Rollback</li>
+ * </ul>
+ * @param <T> Value object class to store the operation specific parameters
+ */
+public class AlterTableExecuteSpec<T> {
+
+ public enum ExecuteOperationType {
+ ROLLBACK
+ }
+
+ private final ExecuteOperationType operationType;
+ private final T operationParams;
+
+ public AlterTableExecuteSpec(ExecuteOperationType type, T value) {
+ this.operationType = type;
+ this.operationParams = value;
+ }
+
+ public ExecuteOperationType getOperationType() {
+ return operationType;
+ }
+
+ public T getOperationParams() {
+ return operationParams;
+ }
+
+ @Override
+ public String toString() {
+ return "operationType=" + operationType.name() + ", " + operationParams;
Review Comment:
nit: We might want to use `MoreObjects` to generate the toString?
Issue Time Tracking
-------------------
Worklog Id: (was: 774439)
Time Spent: 4h (was: 3h 50m)
> Implement Iceberg table rollback feature
> ----------------------------------------
>
> Key: HIVE-26228
> URL: https://issues.apache.org/jira/browse/HIVE-26228
> Project: Hive
> Issue Type: New Feature
> Reporter: László Pintér
> Assignee: László Pintér
> Priority: Major
> Labels: pull-request-available
> Time Spent: 4h
> Remaining Estimate: 0h
>
> We should allow rolling back iceberg table's data to the state at an older
> table snapshot.
> Rollback to the last snapshot before a specific timestamp
> {code:java}
> ALTER TABLE ice_t EXECUTE ROLLBACK('2022-05-12 00:00:00')
> {code}
> Rollback to a specific snapshot ID
> {code:java}
> ALTER TABLE ice_t EXECUTE ROLLBACK(1111);
> {code}
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
