[ 
https://issues.apache.org/jira/browse/HIVE-26102?focusedWorklogId=753986&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-753986
 ]

ASF GitHub Bot logged work on HIVE-26102:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 07/Apr/22 11:24
            Start Date: 07/Apr/22 11:24
    Worklog Time Spent: 10m 
      Work Description: marton-bod commented on code in PR #3131:
URL: https://github.com/apache/hive/pull/3131#discussion_r845020372


##########
ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java:
##########
@@ -7822,9 +7824,18 @@ protected Operator genFileSinkPlan(String dest, QB qb, 
Operator input)
 
     List<ColumnInfo> vecCol = new ArrayList<ColumnInfo>();
 
-    if (updating(dest) || deleting(dest)) {
+    boolean nonNativeAcid = Optional.ofNullable(destinationTable)
+        .map(Table::getStorageHandler)
+        .map(HiveStorageHandler::supportsAcidOperations)
+        .orElse(false);
+    boolean isUpdateDelete = updating(dest) || deleting(dest);
+    if (!nonNativeAcid && isUpdateDelete) {

Review Comment:
   I agree, that's more readable





Issue Time Tracking
-------------------

    Worklog Id:     (was: 753986)
    Time Spent: 11h 20m  (was: 11h 10m)

> Implement DELETE statements for Iceberg tables
> ----------------------------------------------
>
>                 Key: HIVE-26102
>                 URL: https://issues.apache.org/jira/browse/HIVE-26102
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Marton Bod
>            Assignee: Marton Bod
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 11h 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to