sohailshk opened a new pull request, #41088:
URL: https://github.com/apache/beam/pull/41088
Fixes #41087
These raises passed a %-format string and its arguments as separate
exception arguments, so users saw `('... but got %s', 'Row(x=1, y=2)')` instead
of the message. They now format the message.
Added tests for the three detectors (`learn_one` and `score_one`) and for
both `VertexAIModelHandlerJSON._retrieve_endpoint` errors. They fail on master
(the message does not match) and pass with the fix; `ml/anomaly` and
`vertex_ai_inference_test.py`: 109 passed, 2 skipped. yapf and ruff are clean
on the touched files.
The same pattern exists in a few places outside `ml/`; I left those for a
follow-up.
R: @damccorm
--
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]
