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

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

                Author: ASF GitHub Bot
            Created on: 09/Jul/21 08:01
            Start Date: 09/Jul/21 08:01
    Worklog Time Spent: 10m 
      Work Description: klcopp commented on a change in pull request #2460:
URL: https://github.com/apache/hive/pull/2460#discussion_r666753498



##########
File path: 
ql/src/test/org/apache/hadoop/hive/ql/txn/compactor/TestCompactionMetrics.java
##########
@@ -834,19 +833,36 @@ public void testWritesToDisabledCompactionTable() throws 
Exception {
 
   private ShowCompactResponseElement generateElement(long id, String db, 
String table, String partition,
       CompactionType type, String state) {
-    return generateElement(id, db, table, partition, type, state, 
System.currentTimeMillis());
+    return generateElement(id, db, table, partition, type, state, false);
+  }
+
+  private ShowCompactResponseElement generateElement(long id, String db, 
String table, String partition,
+      CompactionType type, String state, boolean manuallyInitiatedCompaction) {
+    return generateElement(id, db, table, partition, type, state, 
manuallyInitiatedCompaction, System.currentTimeMillis());
   }
 
   private ShowCompactResponseElement generateElement(long id, String db, 
String table, String partition,
       CompactionType type, String state, long enqueueTime) {
+    return generateElement(id, db, table, partition, type, state, false, 
enqueueTime);
+  }
+
+  private ShowCompactResponseElement generateElement(long id, String db, 
String table, String partition,
+      CompactionType type, String state, boolean manuallyInitiatedCompaction, 
long enqueueTime) {
     ShowCompactResponseElement element = new ShowCompactResponseElement(db, 
table, type, state);
     element.setId(id);
     element.setPartitionname(partition);
     element.setEnqueueTime(enqueueTime);
 
-    String runtimeId = ServerUtils.hostname() + "-" + 
ThreadLocalRandom.current().nextInt();

Review comment:
       This was generating negative integers, that's why the number of 
initiator and worker hosts was 2 in testUpdateCompactionMetrics




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

    Worklog Id:     (was: 620887)
    Time Spent: 20m  (was: 10m)

> Number of initiator hosts metric should ignore manually initiated compactions
> -----------------------------------------------------------------------------
>
>                 Key: HIVE-25318
>                 URL: https://issues.apache.org/jira/browse/HIVE-25318
>             Project: Hive
>          Issue Type: Sub-task
>    Affects Versions: 4.0.0
>            Reporter: Karen Coppage
>            Assignee: Karen Coppage
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to