potiuk commented on code in PR #27260:
URL: https://github.com/apache/airflow/pull/27260#discussion_r1005071736
##########
dev/breeze/src/airflow_breeze/utils/parallel.py:
##########
@@ -352,6 +354,11 @@ def check_async_run_results(
:param include_success_outputs: include outputs of successful parallel runs
:param poll_time: what's the poll time between checks
:param skip_cleanup: whether to skip cleanup of temporary files.
+ :param print_summary_after_line_matching: if it is not None, the regexp
determines line after which
+ outputs should be printed as summary, so that you do not have to look
at the folded details of
+ the run in CI
+ :param print_summary_also_after_success: if summary is printed, this flag
determines if summaries should
Review Comment:
Actually It will also print summary after failure.
The 'print_summary_after_succes" name might be misleading (Yes i had it like
that initially and when I used it, i realised that it can be misleading so I
added `also`). I could change it to `also_print_summary_after_success` or smth
like that but this is rather small thing :)
--
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]
