Quanlong Huang created IMPALA-12882:
---------------------------------------

             Summary: wait_for_metric_value() doesn't log the current value if 
allow_greater=True
                 Key: IMPALA-12882
                 URL: https://issues.apache.org/jira/browse/IMPALA-12882
             Project: IMPALA
          Issue Type: Bug
          Components: Infrastructure
            Reporter: Quanlong Huang
            Assignee: Quanlong Huang


I saw test failures due to the desired catalog update timeout to be applied on 
the impalad:
{noformat}
-- 2024-02-22 07:40:09,898 INFO     MainThread: Started query 
2b40be0cac7fd641:609759ce00000000
-- 2024-02-22 07:41:10,139 INFO     MainThread: Waiting until events processor 
syncs to event id:40299
-- 2024-02-22 07:41:10,618 DEBUG    MainThread: Metric last-synced-event-id has 
been increased to 40294 but has not yet             reached the desired value: 
40299
-- 2024-02-22 07:41:10,724 DEBUG    MainThread: Metric last-synced-event-id has 
reached the desired value: 40299
-- 2024-02-22 07:41:11,008 DEBUG    MainThread: Found 3 impalad/1 statestored/1 
catalogd process(es)
-- 2024-02-22 07:41:11,046 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:11,053 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:12,054 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:12,061 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:13,062 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:13,067 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:14,067 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:14,075 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:15,077 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:15,082 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:16,082 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:16,086 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:17,087 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:17,096 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:18,097 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:18,113 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:19,114 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:19,120 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:20,122 INFO     MainThread: Getting metric: 
catalog.curr-version from ip-172-31-56-39:25000
-- 2024-02-22 07:41:20,129 INFO     MainThread: Sleeping 1s before next retry.
-- 2024-02-22 07:41:21,130 INFO     MainThread: Metric catalog.curr-version did 
not reach value 8467 in 10s. Failing... {noformat}
Unfortunately, the current value of "catalog.curr-version" is not logged so we 
don't know whether impalad is hanging or moving forward slowly.

The logs come from wait_for_metric_value(). When allow_greater=True, the 
current values are not logged:
{code:python}
      # if allow_greater is True we wait until the metric value becomes >= the 
expected
      # value.
      if allow_greater:
        if value >= expected_value:
          LOG.info("Metric '%s' has reached desired value: %s" % (metric_name, 
value))
          return value
      elif value == expected_value:
        LOG.info("Metric '%s' has reached desired value: %s" % (metric_name, 
value))
        return value
      else:
        LOG.info("Waiting for metric value '%s'%s%s. Current value: %s" %
            (metric_name, '>=' if allow_greater else '=', expected_value, 
value))
      LOG.info("Sleeping %ds before next retry." % interval)
      sleep(interval){code}
https://github.com/apache/impala/blob/54642b2549c38625c868f09815b61170e568cdb9/tests/common/impala_service.py#L129-L138



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

Reply via email to