The GitHub Actions job "Tests" on airflow.git has failed. Run started by GitHub user eladkal (triggered by eladkal).
Head commit for run: b3be18b36f6dd96dd57717d6fa0484fc8f02805e / Daniel Standish <[email protected]> Use None instead of empty data structures when no ElasticSearch logs (#34793) * Use None instead of empty data structures when no ElasticSearch logs Previously, when there were no logs, we would just use [] or {} and continue to process them through functions such as "group by host" and such. But if no logs are found, it's odd to go down those paths. One example of the confusion is the return type in `es_read` was `list | ElasticSearchResponse`. This makes it look like it would either be a list (presumably of logs?) or an ElasticSearchResponse object (presumably also containing logs?). But in reality, the only time list was returned, was when there _were no logs at all!_ This was just to adhere to the contract that we handle "no logs" the same as "logs". _(And it turns out that ElasticSearchResponse magically also behaves like at iterable which is also confusing but that's a separate issue.)_ * use more accurate variable names * more accurate naming * fix when no hits Report URL: https://github.com/apache/airflow/actions/runs/6794693463 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
