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

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

                Author: ASF GitHub Bot
            Created on: 16/Jan/23 11:34
            Start Date: 16/Jan/23 11:34
    Worklog Time Spent: 10m 
      Work Description: veghlaci05 commented on code in PR #3935:
URL: https://github.com/apache/hive/pull/3935#discussion_r1071139423


##########
ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Worker.java:
##########
@@ -315,10 +315,14 @@ protected Boolean findNextCompactionAndExecute(boolean 
collectGenericStats, bool
         return false;
       }
 
-      if (!ci.type.equals(CompactionType.REBALANCE) && ci.numberOfBuckets > 0) 
{
-        if (LOG.isWarnEnabled()) {
-          LOG.warn("Only the REBALANCE compaction accepts the number of 
buckets clause (CLUSTERED INTO {N} BUCKETS). " +
-              "Since the compaction request is {}, it will be ignored.", 
ci.type);
+      boolean insertOnly = AcidUtils.isInsertOnlyTable(table.getParameters());
+      if (LOG.isWarnEnabled() && ci.type.equals(CompactionType.REBALANCE) && 
insertOnly) {

Review Comment:
   No, ci is checked at line 273
   ```
         if (ci == null) {
           return false;
         }
   ```
   type checked at 
org.apache.hadoop.hive.metastore.txn.TxnUtils#dbCompactionType2ThriftType





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

    Worklog Id:     (was: 839379)
    Time Spent: 1.5h  (was: 1h 20m)

> Query based Rebalance compaction on insert-only tables
> ------------------------------------------------------
>
>                 Key: HIVE-26717
>                 URL: https://issues.apache.org/jira/browse/HIVE-26717
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Hive
>            Reporter: László Végh
>            Assignee: László Végh
>            Priority: Major
>              Labels: ACID, compaction, pull-request-available
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to