09Catho opened a new pull request, #41084:
URL: https://github.com/apache/beam/pull/41084

   Fixes #41076
   
   DataflowMetrics._populate_metrics now treats a missing tentative or 
committed update as an absent value instead of indexing both keys. Queries no 
longer raise KeyError when Dataflow reports only one form of a metric.
   
   The regression test covers tentative-only and committed-only user metrics. 
The 2.79.0 bugfix notes include the change.
   
   Validation:
   - PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 python -m pytest 
sdks/python/apache_beam/runners/dataflow/dataflow_metrics_test.py -q: 1 passed, 
5 skipped because GCP dependencies are not installed.
   - ruff check on both changed Python files: passed.
   - python -m py_compile on both changed Python files: passed.
   - git diff --check: passed.
   - ruff format --check reports the two existing files would be reformatted; 
the same result occurs on their untouched master versions.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to